superset-showtime 0.5.1__py3-none-any.whl → 0.5.4__py3-none-any.whl

showtime/__init__.py

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

showtime/core/pull_request.py

@@ -246,6 +246,9 @@ class PullRequest:
                 print(f"✅ Deployment completed - environment running at {show.ip}:8080")
                 self._update_show_labels(show, dry_run_github)
                 
+                # Blue-green cleanup: stop all other environments for this PR
+                cleaned_count = self.stop_previous_environments(show.sha, dry_run_github, dry_run_aws)
+                
                 # Show AWS console URLs for monitoring
                 self._show_service_urls(show)
 
@@ -278,6 +281,9 @@ class PullRequest:
                 print(f"✅ Rolling update completed - new environment at {new_show.ip}:8080")
                 self._update_show_labels(new_show, dry_run_github)
                 
+                # Blue-green cleanup: stop all other environments for this PR
+                cleaned_count = self.stop_previous_environments(new_show.sha, dry_run_github, dry_run_aws)
+                
                 # Show AWS console URLs for monitoring
                 self._show_service_urls(new_show)
 
@@ -389,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
@@ -429,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}")
@@ -580,6 +589,17 @@ class PullRequest:
         for old_status_label in sha_status_labels:
             get_github().remove_label(self.pr_number, old_status_label)
 
+        # For running environments, ensure only ONE active pointer exists
+        if show.status == "running":
+            # Remove ALL existing active pointers (there should only be one)
+            existing_active_pointers = [
+                label for label in self.labels 
+                if label.startswith("🎪 🎯 ")
+            ]
+            for old_pointer in existing_active_pointers:
+                print(f"🎯 Removing old active pointer: {old_pointer}")
+                get_github().remove_label(self.pr_number, old_pointer)
+
         # Now do normal differential updates - only for this SHA
         current_sha_labels = {
             label for label in self.labels 
@@ -616,3 +636,51 @@ class PullRequest:
         print(f"📝 Logs: {urls['logs']}")
         print(f"📊 Service: {urls['service']}")
         print("")
+
+    def stop_previous_environments(self, keep_sha: str, dry_run_github: bool = False, dry_run_aws: bool = False) -> int:
+        """Stop all environments except the specified SHA (blue-green cleanup)
+        
+        Args:
+            keep_sha: SHA of environment to keep running
+            dry_run_github: Skip GitHub label operations  
+            dry_run_aws: Skip AWS operations
+            
+        Returns:
+            Number of environments stopped
+        """
+        # CRITICAL: Refresh to get current shows including newly created one
+        self.refresh_labels()
+        
+        stopped_count = 0
+        
+        for show in self.shows:
+            if show.sha != keep_sha:
+                print(f"🧹 Cleaning up old environment: {show.sha} ({show.status})")
+                try:
+                    show.stop(dry_run_github=dry_run_github, dry_run_aws=dry_run_aws)
+                    
+                    # Remove labels for this old environment
+                    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:
+                            try:
+                                get_github().remove_label(self.pr_number, label)
+                            except Exception as e:
+                                print(f"⚠️ Failed to remove label {label}: {e}")
+                    
+                    stopped_count += 1
+                    print(f"✅ Stopped environment {show.sha}")
+                    
+                except Exception as e:
+                    print(f"❌ Failed to stop environment {show.sha}: {e}")
+                    
+        if stopped_count > 0:
+            print(f"🧹 Blue-green cleanup: stopped {stopped_count} old environments")
+            # Refresh labels after cleanup
+            if not dry_run_github:
+                self.refresh_labels()
+        else:
+            print("ℹ️ No old environments to clean up")
+            
+        return stopped_count

showtime/core/show.py

@@ -165,17 +165,17 @@ class Show:
 
         # Detect if running in CI environment
         is_ci = bool(os.getenv("GITHUB_ACTIONS") or os.getenv("CI"))
-        
+
         # Build command without final path
         cmd = [
             "docker",
-            "buildx", 
+            "buildx",
             "build",
             "--push",
             "--platform",
             "linux/amd64",
             "--target",
-            "dev",
+            "showtime",
             "--build-arg",
             "INCLUDE_CHROMIUM=false",
             "--build-arg",
@@ -190,7 +190,7 @@ class Show:
             cmd.extend([
                 "--cache-from",
                 "type=registry,ref=apache/superset-cache:showtime",
-                "--cache-to", 
+                "--cache-to",
                 "type=registry,mode=max,ref=apache/superset-cache:showtime",
             ])
             print("🐳 CI environment: Using full registry caching")
@@ -204,7 +204,7 @@ class Show:
 
         # Add --load only when explicitly requested for local testing
         force_load = os.getenv("DOCKER_LOAD", "false").lower() == "true"
-        
+
         if force_load:
             cmd.append("--load")
             print("🐳 Will load image to local Docker daemon (DOCKER_LOAD=true)")

{superset_showtime-0.5.1.dist-info → superset_showtime-0.5.4.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: superset-showtime
-Version: 0.5.1
+Version: 0.5.4
 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.1.dist-info → superset_showtime-0.5.4.dist-info}/RECORD

@@ -1,4 +1,4 @@
-showtime/__init__.py,sha256=5ryBVT5km-Ec4J3VGcfmM-qk9YlhGYz1UuGdv8d9vZg,448
+showtime/__init__.py,sha256=v4dfA5K-c1pUPNrB3iXwD3jjOyurED35BMm4yRTnJ24,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=PnDTvC57YDNxz2XwhZi0Zf5FHzeYqJyfdlCmlI85GNo,24071
-showtime/core/show.py,sha256=P2LCVtzG6P2HXR9_oYX3OZcDeQJpngee4sw-iv62vac,9714
+showtime/core/pull_request.py,sha256=DMuNkoWZETg55CJuWsyv58UOpgkUWeT-XJbA_97OKjA,27229
+showtime/core/show.py,sha256=FpxDm52LASCJvf8UF998AtNiVzfdYIwNEsPAsOAAwL0,9701
 showtime/data/ecs-task-definition.json,sha256=2acmqoF-3CxaBJP_VDkMMpG_U2RI4VPk1JvFOprMFyc,2098
-superset_showtime-0.5.1.dist-info/METADATA,sha256=JjLoKU2jo9fmxNwFW34JDRTwZMlTaqpDgcP38c9akms,12052
-superset_showtime-0.5.1.dist-info/WHEEL,sha256=qtCwoSJWgHk21S1Kb4ihdzI2rlJ1ZKaIurTj_ngOhyQ,87
-superset_showtime-0.5.1.dist-info/entry_points.txt,sha256=rDW7oZ57mqyBUS4N_3_R7bZNGVHB-104jwmY-hHC_ck,85
-superset_showtime-0.5.1.dist-info/RECORD,,
+superset_showtime-0.5.4.dist-info/METADATA,sha256=vwZMCk_jL9X5_ZBPi8Gy-jYBZKhFkICinHC-eop0qsY,12052
+superset_showtime-0.5.4.dist-info/WHEEL,sha256=qtCwoSJWgHk21S1Kb4ihdzI2rlJ1ZKaIurTj_ngOhyQ,87
+superset_showtime-0.5.4.dist-info/entry_points.txt,sha256=rDW7oZ57mqyBUS4N_3_R7bZNGVHB-104jwmY-hHC_ck,85
+superset_showtime-0.5.4.dist-info/RECORD,,