Merge pull request #4833 from hashicorp/backport/missing-efs-panic/su…

…btly-true-chipmunk This pull request was automerged via backport-assistant
hashicorp · Jul 6, 2023 · 51fbee1 · 51fbee1
2 parents b00ef1c + d770db6
commit 51fbee1
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/.changelog/4829.txt b/.changelog/4829.txt
@@ -0,0 +1,3 @@
+```release-note:bug
+runneruninstall/aws-ecs: Fix panic when uninstalling ECS runner after failing to find EFS
+```
diff --git a/internal/runnerinstall/ecs.go b/internal/runnerinstall/ecs.go
@@ -397,7 +397,7 @@ func (i *ECSRunnerInstaller) Uninstall(ctx context.Context, opts *InstallOpts) e
 		}
 	}
 
-	if *fileSystemId == "" {
+	if fileSystemId == nil || *fileSystemId == "" {
 		s.Update("File system with tag key `runner-id` and value " + opts.Id + " not detected, skipping deletion")
 		s.Done()
 		return nil