Skip to content

Commit c32367b

Browse files
trifo13claude
andcommitted
PRODENG-3446: fix AWS name-length overflow after rename
smokeConfig.Name drives the Terraform stack name as "smoke-{Name}-{5-char-random}"; Terraform then appends suffixes like "-mke-kube" for target groups, capping the total at 32 chars. "airgapped-multi-hop" (19 chars) overflows that limit — revert to "airgap-mhop" (11 chars, same limit as the original "airgappedup"). Add a comment explaining why the short name is necessary despite the longer CI label. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
1 parent 597481f commit c32367b

1 file changed

Lines changed: 5 additions & 1 deletion

File tree

test/smoke/airgapped_multi_hop_upgrade_test.go

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -626,7 +626,11 @@ func TestAirgappedMultiHopUpgrade(t *testing.T) {
626626

627627
runAirgappedMultiHopUpgradeTest(t, airgapUpgradeConfig{
628628
Base: smokeConfig{
629-
Name: "airgapped-multi-hop",
629+
// NOTE: AWS LB/target-group names are limited to 32 chars. The stack
630+
// name is "smoke-{Name}-{5-char-random}", and Terraform appends suffixes
631+
// like "-mke-kube" (9 chars). That caps len(Name) at 11. Keep this
632+
// short even though the CI label is "smoke-airgapped-multi-hop".
633+
Name: "airgap-mhop",
630634
MCRChannel: "stable-25.0",
631635
MKEVersion: "3.8.8",
632636
MSRVersion: "2.9.27",

0 commit comments

Comments
 (0)