fix: Fix missing tag for SSM parameter EGRESS_IP (#349)

uktrade · Feb 28, 2025 · 8483746 · 8483746
1 parent d146e73
commit 8483746
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 1 deletion.
diff --git a/vpc/main.tf b/vpc/main.tf
@@ -118,7 +118,13 @@ resource "aws_ssm_parameter" "combined_nat_gateway_eips" {
   name  = "/${var.arg_name}/EGRESS_IPS"
   type  = "String"
   value = join(",", local.nat_gateway_eips)
-  tags  = local.tags
+  tags = merge(
+    local.tags,
+    {
+      "copilot-application" = "__all__"
+    }
+  )
+
 }
 
 

diff --git a/vpc/tests/unit.tftest.hcl b/vpc/tests/unit.tftest.hcl
@@ -64,6 +64,12 @@ run "aws_vpc_unit_test" {
   }
 
   # aws_ssm_parameter.combined_nat_gateway_eips.value cannot be tested on a plan
+
+  assert {
+    condition     = lookup(aws_ssm_parameter.combined_nat_gateway_eips.tags, "copilot-application", "") == "__all__"
+    error_message = "Tag 'copilot-application' should be set to '__all__'"
+  }
+
 }
 
 run "aws_security_group_unit_test" {