fix: Correct metadata options loop condition due to variable definition defaults (#3490)

bryantbiggs · web-flow · commit b40968a503f1 · 2025-08-24T10:08:35.000-05:00
diff --git a/modules/eks-managed-node-group/main.tf b/modules/eks-managed-node-group/main.tf
@@ -224,7 +224,7 @@ resource "aws_launch_template" "this" {
   }
 
   dynamic "metadata_options" {
-    for_each = var.metadata_options != null ? [var.metadata_options] : []
+    for_each = [var.metadata_options]
 
     content {
       http_endpoint               = metadata_options.value.http_endpoint
diff --git a/modules/self-managed-node-group/main.tf b/modules/self-managed-node-group/main.tf
@@ -342,7 +342,7 @@ resource "aws_launch_template" "this" {
   }
 
   dynamic "metadata_options" {
-    for_each = var.metadata_options != null ? [var.metadata_options] : []
+    for_each = [var.metadata_options]
 
     content {
       http_endpoint               = metadata_options.value.http_endpoint

Original file line number	Diff line number	Diff line change
`@@ -224,7 +224,7 @@ resource "aws_launch_template" "this" {`
`224`	`224`	`}`
`225`	`225`
`226`	`226`	`dynamic "metadata_options" {`
`227`		`- for_each = var.metadata_options != null ? [var.metadata_options] : []`
	`227`	`+ for_each = [var.metadata_options]`
`228`	`228`
`229`	`229`	`content {`
`230`	`230`	`http_endpoint = metadata_options.value.http_endpoint`
Original file line number	Diff line number	Diff line change
`@@ -342,7 +342,7 @@ resource "aws_launch_template" "this" {`
`342`	`342`	`}`
`343`	`343`
`344`	`344`	`dynamic "metadata_options" {`
`345`		`- for_each = var.metadata_options != null ? [var.metadata_options] : []`
	`345`	`+ for_each = [var.metadata_options]`
`346`	`346`
`347`	`347`	`content {`
`348`	`348`	`http_endpoint = metadata_options.value.http_endpoint`