Nulls the compute_config entirely when auto mode is disabled

lorengordon · lorengordon · commit c91f8a6e32bd · 2025-02-21T18:02:04.000-08:00
The API will return an error if compute_config.enabled is set to false, when every other required option for auto mode is also not provided. ``` │ Error: creating EKS Cluster (test-minimum-inputs-kuneic): operation error EKS: CreateCluster, https response error StatusCode: 400, RequestID: 573c664d-68c6-463c-b117-ba12722ebe96, InvalidParameterException: For EKS Auto Mode, please ensure that all required configs, including computeConfig, kubernetesNetworkConfig, and blockStorage are all either fully enabled or fully disabled. ``` Seems like an odd API choice, but that's how it works. Fixes #3307
diff --git a/main.tf b/main.tf
@@ -53,7 +53,7 @@ resource "aws_eks_cluster" "this" {
   }
 
   dynamic "compute_config" {
-    for_each = length(var.cluster_compute_config) > 0 ? [var.cluster_compute_config] : []
+    for_each = local.auto_mode_enabled ? [var.cluster_compute_config] : []
 
     content {
       enabled       = local.auto_mode_enabled

Original file line number	Diff line number	Diff line change
`@@ -53,7 +53,7 @@ resource "aws_eks_cluster" "this" {`
`53`	`53`	`}`
`54`	`54`
`55`	`55`	`dynamic "compute_config" {`
`56`		`- for_each = length(var.cluster_compute_config) > 0 ? [var.cluster_compute_config] : []`
	`56`	`+ for_each = local.auto_mode_enabled ? [var.cluster_compute_config] : []`
`57`	`57`
`58`	`58`	`content {`
`59`	`59`	`enabled = local.auto_mode_enabled`