Fix support for AMIs with parentheses in the name

Sbozzolo · Sbozzolo · commit 4ad8040b3603 · 2025-10-29T13:42:59.000-07:00
Some AMIs, like the `AWS Deep Learning Base GPU AMI (Ubuntu 24.04)`,
have parentheses in the name. Parentheses are not allowed in SSM tags,
leading to an error. This commit strips the parentheses out of the name.
diff --git a/modules/runners/main.tf b/modules/runners/main.tf
@@ -102,7 +102,8 @@ resource "aws_ssm_parameter" "runner_ami_id" {
   tags = merge(
     local.tags,
     {
-      "ghr:ami_name" = data.aws_ami.runner.name
+      # Remove parentheses from AMI name to comply with SSM tag constraints
+      "ghr:ami_name" = replace(data.aws_ami.runner.name, "/[()]/", "")
     },
     {
       "ghr:ami_creation_date" = data.aws_ami.runner.creation_date

Original file line number	Diff line number	Diff line change
`@@ -102,7 +102,8 @@ resource "aws_ssm_parameter" "runner_ami_id" {`
`102`	`102`	`tags = merge(`
`103`	`103`	`local.tags,`
`104`	`104`	`{`
`105`		`- "ghr:ami_name" = data.aws_ami.runner.name`
	`105`	`+ # Remove parentheses from AMI name to comply with SSM tag constraints`
	`106`	`+ "ghr:ami_name" = replace(data.aws_ami.runner.name, "/[()]/", "")`
`106`	`107`	`},`
`107`	`108`	`{`
`108`	`109`	`"ghr:ami_creation_date" = data.aws_ami.runner.creation_date`