superset-showtime 0.5.0__tar.gz → 0.5.1__tar.gz

{superset_showtime-0.5.0 → superset_showtime-0.5.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: superset-showtime
-Version: 0.5.0
+Version: 0.5.1
 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.0 → superset_showtime-0.5.1}/showtime/__init__.py

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

{superset_showtime-0.5.0 → superset_showtime-0.5.1}/showtime/cli.py

@@ -722,7 +722,7 @@ def aws_cleanup(
                 parts = pr_match.split("-")
                 if len(parts) >= 2:
                     pr_number = int(parts[0])
-                    success = aws.delete_environment(service.replace("-service", ""), pr_number)
+                    success = aws.delete_environment(service, pr_number)
                     if success:
                         p(f"✅ Deleted {service}")
                         deleted_count += 1

{superset_showtime-0.5.0 → superset_showtime-0.5.1}/showtime/core/aws.py

@@ -175,36 +175,62 @@ class AWSInterface:
 
     def delete_environment(self, service_name: str, pr_number: int) -> bool:
         """
-        Delete ephemeral environment - replicates cleanup GHA logic
+        Delete ephemeral environment with proper verification
 
         Steps:
-        1. Check if ECS service exists and is active
-        2. Delete ECS service with --force
+        1. Check if ECS service exists 
+        2. Delete ECS service with force and wait for completion
         3. Delete ECR image tag
+        4. Verify deletion completed
         """
         try:
-            # Step 1: Check if service exists and is active
-            if not self._service_exists(service_name):
-                return True  # Already deleted
+            ecs_service_name = f"{service_name}-service" if not service_name.endswith("-service") else service_name
+            print(f"🗑️ Deleting ECS service: {ecs_service_name}")
+            
+            # Step 1: Check if service exists
+            if not self._service_exists(ecs_service_name):
+                print(f"✅ Service {ecs_service_name} already deleted")
+                return True
+
+            # Step 2: Delete ECS service (force delete) and wait
+            print(f"☁️ Initiating ECS service deletion...")
+            delete_response = self.ecs_client.delete_service(
+                cluster=self.cluster, 
+                service=ecs_service_name, 
+                force=True
+            )
+            print(f"🔄 Delete initiated: {delete_response.get('service', {}).get('status', 'unknown')}")
 
-            # Step 2: Delete ECS service (force delete)
-            self.ecs_client.delete_service(cluster=self.cluster, service=service_name, force=True)
+            # Step 3: Wait for deletion to complete (crucial!)
+            print(f"⏳ Waiting for service deletion to complete...")
+            deletion_success = self._wait_for_service_deletion(ecs_service_name, timeout_minutes=10)
+            
+            if not deletion_success:
+                print(f"⚠️ Service deletion timeout - service may still exist")
+                return False
 
-            # Step 3: Delete ECR image tag
-            # Extract SHA from service name: pr-1234-abc123f → abc123f
-            sha = service_name.split("-")[-1]
-            image_tag = f"pr-{pr_number}-{sha}"
+            # Step 4: Delete ECR image tag
+            print(f"🐳 Cleaning up Docker image...")
+            # Fix SHA extraction: pr-34831-ac533ec-service → ac533ec
+            # Remove "pr-" prefix and "-service" suffix, then get SHA (last part)
+            base_name = service_name.replace("pr-", "").replace("-service", "")
+            parts = base_name.split("-")
+            sha = parts[-1] if len(parts) > 1 else base_name  # Last part is SHA
+            image_tag = f"pr-{pr_number}-{sha}-ci"
 
             try:
                 self.ecr_client.batch_delete_image(
                     repositoryName=self.repository, imageIds=[{"imageTag": image_tag}]
                 )
+                print(f"✅ Deleted ECR image: {image_tag}")
             except self.ecr_client.exceptions.ImageNotFoundException:
-                pass  # Image already deleted
+                print(f"ℹ️ ECR image {image_tag} already deleted")
 
+            print(f"✅ Environment {service_name} fully deleted")
             return True
 
         except Exception as e:
+            print(f"❌ AWS deletion failed: {e}")
             raise AWSError(
                 message=str(e), operation="delete_environment", resource=service_name
             ) from e

{superset_showtime-0.5.0 → superset_showtime-0.5.1}/showtime/core/pull_request.py

@@ -450,17 +450,31 @@ class PullRequest:
 
         # 2. Remove trigger labels (atomic operation)
         trigger_labels = [label for label in self.labels if "showtime-trigger-" in label]
-        for trigger_label in trigger_labels:
-            get_github().remove_label(self.pr_number, trigger_label)
+        if trigger_labels:
+            print(f"🏷️ Removing trigger labels: {trigger_labels}")
+            for trigger_label in trigger_labels:
+                get_github().remove_label(self.pr_number, trigger_label)
+        else:
+            print("🏷️ No trigger labels to remove")
 
         # 3. Set building state immediately (claim the PR)
         if action in ["create_environment", "rolling_update", "auto_sync"]:
             building_show = self._create_new_show(target_sha)
             building_show.status = "building"
+            
             # Update labels to reflect building state
+            print(f"🏷️ Removing existing circus labels...")
             get_github().remove_circus_labels(self.pr_number)
-            for label in building_show.to_circus_labels():
-                get_github().add_label(self.pr_number, label)
+            
+            new_labels = building_show.to_circus_labels()
+            print(f"🏷️ Creating new labels: {new_labels}")
+            for label in new_labels:
+                try:
+                    get_github().add_label(self.pr_number, label)
+                    print(f"  ✅ Added: {label}")
+                except Exception as e:
+                    print(f"  ❌ Failed to add {label}: {e}")
+                    raise
 
         return True
 

{superset_showtime-0.5.0 → superset_showtime-0.5.1}/showtime/core/show.py

@@ -202,16 +202,14 @@ class Show:
             ])
             print("🐳 Local environment: Using cache-from only (no export)")
 
-        # Add --load only when building for native architecture or explicitly requested
-        # Intel Mac/Linux can load linux/amd64, Apple Silicon cannot
-        native_x86 = platform.machine() in ("x86_64", "AMD64")
+        # Add --load only when explicitly requested for local testing
         force_load = os.getenv("DOCKER_LOAD", "false").lower() == "true"
         
-        if native_x86 or force_load:
+        if force_load:
             cmd.append("--load")
-            print("🐳 Will load image to local Docker daemon (native x86_64 platform)")
+            print("🐳 Will load image to local Docker daemon (DOCKER_LOAD=true)")
         else:
-            print("🐳 Cross-platform build - pushing to registry only (no local load)")
+            print("🐳 Push-only build (no local load) - faster for CI/deployment")
 
         # Add build context path last
         cmd.append(".")