fix: Regression for NLB container name (#291)

adamantike · gberenice · web-flow · commit ca3040fccb15 · 2024-10-15T19:40:11.000+03:00
A regression was introduced in a previous commit, where the NLB
container name is being set to the value of the `alb_container_name`
variable, instead of using `nlb_container_name`.

Co-authored-by: Veronika Gnilitska &lt;30597968+gberenice@users.noreply.github.com&gt;
diff --git a/main.tf b/main.tf
@@ -120,7 +120,7 @@ locals {
     target_group_arn = module.alb_ingress.target_group_arn
   }
   nlb = {
-    container_name   = var.alb_container_name != null ? var.alb_container_name : module.this.id
+    container_name   = var.nlb_container_name != null ? var.nlb_container_name : module.this.id
     container_port   = var.nlb_container_port
     elb_name         = null
     target_group_arn = var.nlb_ingress_target_group_arn

Original file line number	Diff line number	Diff line change
`@@ -120,7 +120,7 @@ locals {`
`120`	`120`	`target_group_arn = module.alb_ingress.target_group_arn`
`121`	`121`	`}`
`122`	`122`	`nlb = {`
`123`		`- container_name = var.alb_container_name != null ? var.alb_container_name : module.this.id`
	`123`	`+ container_name = var.nlb_container_name != null ? var.nlb_container_name : module.this.id`
`124`	`124`	`container_port = var.nlb_container_port`
`125`	`125`	`elb_name = null`
`126`	`126`	`target_group_arn = var.nlb_ingress_target_group_arn`