Skip to content

Commit

Permalink
🐛 fix([variables.tf]): validation conditions can only reference varia…
Browse files Browse the repository at this point in the history
…ble under test
  • Loading branch information
morga471 committed Sep 4, 2025
1 parent 98f8130 commit 07dab85
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 12 deletions.
6 changes: 3 additions & 3 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -173,9 +173,9 @@ efs-csi-controller 0 5m
| <a name="input_cluster_version"></a> [cluster\_version](#input\_cluster\_version) | Kubernetes version to use for the EKS cluster | `string` | n/a | yes |
| <a name="input_eks_instance_disk_size"></a> [eks\_instance\_disk\_size](#input\_eks\_instance\_disk\_size) | Size of the EKS node disk in GB | `number` | `80` | no |
| <a name="input_eks_instance_types"></a> [eks\_instance\_types](#input\_eks\_instance\_types) | List of EC2 instance types for the EKS node group | `list(string)` | <pre>[<br/> "t3a.medium"<br/>]</pre> | no |
| <a name="input_eks_ng_desired_size"></a> [eks\_ng\_desired\_size](#input\_eks\_ng\_desired\_size) | Desired size of the EKS node group | `number` | `4` | no |
| <a name="input_eks_ng_max_size"></a> [eks\_ng\_max\_size](#input\_eks\_ng\_max\_size) | Maximum size of the EKS node group | `number` | `15` | no |
| <a name="input_eks_ng_min_size"></a> [eks\_ng\_min\_size](#input\_eks\_ng\_min\_size) | Minimum size of the EKS node group | `number` | `4` | no |
| <a name="input_eks_ng_desired_size"></a> [eks\_ng\_desired\_size](#input\_eks\_ng\_desired\_size) | Desired size of the EKS node group | `number` | `2` | no |
| <a name="input_eks_ng_max_size"></a> [eks\_ng\_max\_size](#input\_eks\_ng\_max\_size) | Maximum size of the EKS node group | `number` | `2` | no |
| <a name="input_eks_ng_min_size"></a> [eks\_ng\_min\_size](#input\_eks\_ng\_min\_size) | Minimum size of the EKS node group | `number` | `2` | no |
| <a name="input_enable_cluster_creator_admin_permissions"></a> [enable\_cluster\_creator\_admin\_permissions](#input\_enable\_cluster\_creator\_admin\_permissions) | Grant admin permissions to the cluster creator | `bool` | `true` | no |
| <a name="input_subnets_name"></a> [subnets\_name](#input\_subnets\_name) | Name pattern for subnets to be used by EKS cluster | `string` | `"*-container-*"` | no |
| <a name="input_tags"></a> [tags](#input\_tags) | Additional tags to apply to all resources | `map(string)` | `{}` | no |
Expand Down
14 changes: 5 additions & 9 deletions variables.tf
Original file line number Diff line number Diff line change
Expand Up @@ -82,7 +82,7 @@ variable "eks_instance_types" {
variable "eks_ng_min_size" {
description = "Minimum size of the EKS node group"
type = number
default = 4
default = 2
validation {
condition = var.eks_ng_min_size >= 1
error_message = "Minimum node group size must be at least 1."
Expand All @@ -92,11 +92,7 @@ variable "eks_ng_min_size" {
variable "eks_ng_desired_size" {
description = "Desired size of the EKS node group"
type = number
default = 4
validation {
condition = var.eks_ng_desired_size >= var.eks_ng_min_size && var.eks_ng_desired_size <= var.eks_ng_max_size
error_message = "Desired size must be between minimum and maximum sizes."
}
default = 2
validation {
condition = var.eks_ng_desired_size >= 1
error_message = "Desired size must be at least 1."
Expand All @@ -106,10 +102,10 @@ variable "eks_ng_desired_size" {
variable "eks_ng_max_size" {
description = "Maximum size of the EKS node group"
type = number
default = 15
default = 2
validation {
condition = var.eks_ng_max_size >= var.eks_ng_min_size
error_message = "Maximum node group size must be greater than or equal to minimum size."
condition = var.eks_ng_max_size >= 1
error_message = "Maximum node group size must be at least 1."
}
}

Expand Down

0 comments on commit 07dab85

Please sign in to comment.