superset-showtime 0.5.3__py3-none-any.whl → 0.5.6__py3-none-any.whl

showtime/__init__.py

@@ -4,7 +4,7 @@
 Circus tent emoji state tracking for Apache Superset ephemeral environments.
 """
 
-__version__ = "0.5.3"
+__version__ = "0.5.6"
 __author__ = "Maxime Beauchemin"
 __email__ = "maximebeauchemin@gmail.com"
 

showtime/core/pull_request.py

@@ -395,6 +395,9 @@ class PullRequest:
 
     def _determine_action(self, target_sha: str) -> str:
         """Determine what sync action is needed based on target SHA state"""
+        # CRITICAL: Get fresh labels before any decisions
+        self.refresh_labels()
+        
         target_sha_short = target_sha[:7]  # Ensure we're working with short SHA
         
         # Get the specific show for the target SHA
@@ -435,20 +438,20 @@ class PullRequest:
 
     def _atomic_claim(self, target_sha: str, action: str, dry_run: bool = False) -> bool:
         """Atomically claim this PR for the current job based on target SHA state"""
+        # CRITICAL: Get fresh labels before any decisions
+        self.refresh_labels()
+        
         target_sha_short = target_sha[:7]
         target_show = self.get_show_by_sha(target_sha_short)
         
-        # 1. Validate current state allows this action for target SHA
+        # 1. Validate current state allows this action for target SHA  
         if action in ["create_environment", "rolling_update", "auto_sync"]:
             if target_show and target_show.status in [
                 "building",
                 "built", 
-                "deploying",
+                "deploying", 
             ]:
-                return False  # Target SHA already in progress
-            
-            # Allow actions on failed, running, or non-existent target SHAs
-            return True
+                return False  # Target SHA already in progress - ONLY conflict case returns
 
         if dry_run:
             print(f"🎪 [DRY-RUN] Would atomically claim PR for {action}")
@@ -645,6 +648,7 @@ class PullRequest:
         Returns:
             Number of environments stopped
         """
+        # Note: Labels should be fresh from recent _update_show_labels() call
         stopped_count = 0
         
         for show in self.shows:
@@ -653,11 +657,16 @@ class PullRequest:
                 try:
                     show.stop(dry_run_github=dry_run_github, dry_run_aws=dry_run_aws)
                     
-                    # Remove labels for this old environment
+                    # Remove ONLY existing labels for this old environment (not theoretical ones)
                     if not dry_run_github:
-                        old_labels = show.to_circus_labels()
-                        print(f"🏷️ Removing labels for {show.sha}: {len(old_labels)} labels")
-                        for label in old_labels:
+                        existing_labels = [
+                            label for label in self.labels 
+                            if label.startswith(f"🎪 {show.sha} ") or 
+                               label == f"🎪 🎯 {show.sha}" or 
+                               label == f"🎪 🏗️ {show.sha}"
+                        ]
+                        print(f"🏷️ Removing existing labels for {show.sha}: {existing_labels}")
+                        for label in existing_labels:
                             try:
                                 get_github().remove_label(self.pr_number, label)
                             except Exception as e:

showtime/data/ecs-task-definition.json

@@ -67,5 +67,5 @@
         "FARGATE"
     ],
     "cpu": "512",
-    "memory": "1024"
+    "memory": "2048"
 }

{superset_showtime-0.5.3.dist-info → superset_showtime-0.5.6.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: superset-showtime
-Version: 0.5.3
+Version: 0.5.6
 Summary: 🎪 Apache Superset ephemeral environment management with circus tent emoji state tracking
 Project-URL: Homepage, https://github.com/apache/superset-showtime
 Project-URL: Documentation, https://superset-showtime.readthedocs.io/

{superset_showtime-0.5.3.dist-info → superset_showtime-0.5.6.dist-info}/RECORD

@@ -1,4 +1,4 @@
-showtime/__init__.py,sha256=HKR0Quvgeabq-6nUgJV_sgWCHcm5jzwuQR_uOoXJ9VQ,448
+showtime/__init__.py,sha256=-Ztgg8h_575Pgg976_q2WLHdhCe_pDlZfkn4C2DCjws,448
 showtime/__main__.py,sha256=EVaDaTX69yIhCzChg99vqvFSCN4ELstEt7Mpb9FMZX8,109
 showtime/cli.py,sha256=cQB5kH-XWFX3MhsxRaChVdjVe_I_7kBVyFY4fQrHPWA,30542
 showtime/core/__init__.py,sha256=54hbdFNGrzuNMBdraezfjT8Zi6g221pKlJ9mREnKwCw,34
@@ -7,10 +7,10 @@ showtime/core/emojis.py,sha256=MHEDuPIdfNiop4zbNLuviz3eY05QiftYSHHCVbkfKhw,2129
 showtime/core/github.py,sha256=uETvKDO2Yhpqg3fxLtrKaCuZR3b-1LVmgnf5aLcqrAQ,9988
 showtime/core/github_messages.py,sha256=MfgwCukrEsWWesMsuL8saciDgP4nS-gijzu8DXr-Alg,7450
 showtime/core/label_colors.py,sha256=efhbFnz_3nqEnEqmgyF6_hZbxtCu_fmb68BIIUpSsnk,3895
-showtime/core/pull_request.py,sha256=X0LU717-Bzs5TSmD36DMaNQ9KfpwXMJ74OWr2aBj7X0,27000
+showtime/core/pull_request.py,sha256=V1TF_KYYqigQHWndRL72CRfVcpOSo_IOh_JHu7HJzVE,27512
 showtime/core/show.py,sha256=FpxDm52LASCJvf8UF998AtNiVzfdYIwNEsPAsOAAwL0,9701
-showtime/data/ecs-task-definition.json,sha256=2acmqoF-3CxaBJP_VDkMMpG_U2RI4VPk1JvFOprMFyc,2098
-superset_showtime-0.5.3.dist-info/METADATA,sha256=nP5JM98Vi0uUgEYBn2lVxsXRVvb4_CuK9U-WgbiBlZE,12052
-superset_showtime-0.5.3.dist-info/WHEEL,sha256=qtCwoSJWgHk21S1Kb4ihdzI2rlJ1ZKaIurTj_ngOhyQ,87
-superset_showtime-0.5.3.dist-info/entry_points.txt,sha256=rDW7oZ57mqyBUS4N_3_R7bZNGVHB-104jwmY-hHC_ck,85
-superset_showtime-0.5.3.dist-info/RECORD,,
+showtime/data/ecs-task-definition.json,sha256=P7NQljXZTOPW1fWdehGrwJGAh5vdFK9-LzGT59q1PU0,2098
+superset_showtime-0.5.6.dist-info/METADATA,sha256=1zFfoAQUM_UnA9u_C6gK64zeXfPU4TXQnACMVh7lZxQ,12052
+superset_showtime-0.5.6.dist-info/WHEEL,sha256=qtCwoSJWgHk21S1Kb4ihdzI2rlJ1ZKaIurTj_ngOhyQ,87
+superset_showtime-0.5.6.dist-info/entry_points.txt,sha256=rDW7oZ57mqyBUS4N_3_R7bZNGVHB-104jwmY-hHC_ck,85
+superset_showtime-0.5.6.dist-info/RECORD,,