fix: handle destroy with missing security group (#381)

This PR makes the following changes: * Destroying cluser even if security group is missing (for example in case of ungraceful stop during a previous destroy) I tested this PR by hand and against incomplete cluster (missing security group) Co-authored-by: Nicholas Chammas <[email protected]>
nchammas · Dec 13, 2024 · 3cce613 · 3cce613
1 parent a11946d
commit 3cce613
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/flintrock/ec2.py b/flintrock/ec2.py
@@ -195,7 +195,10 @@ def destroy(self):
             region=self.region,
             cluster_name=self.name,
         )
-        cluster_group.delete()
+        # Cluster group might already have been killed if a destroy was ungracefully stopped during
+        # a previous execution.
+        if cluster_group:
+            cluster_group.delete()
 
         (ec2.instances
             .filter(