fix(runners): support for AMIs with parentheses in the name (#4856)

Sbozzolo · npalm · web-flow · commit 2904a10e0b0d · 2025-11-23T11:00:49.000+01:00
Some AMIs, like the `AWS Deep Learning Base GPU AMI (Ubuntu 24.04)`,
have parentheses in the name. Parentheses are not allowed in AWS tags,
leading to an error. This commit strips the parentheses out of the name.

Co-authored-by: Niek Palm &lt;npalm@users.noreply.github.com&gt;
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 AWS 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 AWS 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`