superset-showtime 0.2.9__tar.gz → 0.3.1__tar.gz

{superset_showtime-0.2.9 → superset_showtime-0.3.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: superset-showtime
-Version: 0.2.9
+Version: 0.3.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.2.9 → superset_showtime-0.3.1}/showtime/__init__.py

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

{superset_showtime-0.2.9 → superset_showtime-0.3.1}/showtime/cli.py

@@ -330,21 +330,36 @@ def _build_docker_image(pr_number: int, sha: str, dry_run: bool = False) -> bool
 
     try:
         console.print(f"🎪 Running: {' '.join(cmd)}")
-        result = subprocess.run(cmd, capture_output=True, text=True, timeout=1800)  # 30 min timeout
+        console.print("🎪 Streaming Docker build output...")
+
+        # Stream output in real-time for better user experience
+        process = subprocess.Popen(
+            cmd,
+            stdout=subprocess.PIPE,
+            stderr=subprocess.STDOUT,
+            text=True,
+            bufsize=1,
+            universal_newlines=True,
+        )
+
+        # Stream output line by line
+        for line in process.stdout:
+            console.print(f"🐳 {line.rstrip()}")
 
-        if result.returncode == 0:
+        # Wait for completion with timeout
+        try:
+            return_code = process.wait(timeout=3600)  # 60 min timeout
+        except subprocess.TimeoutExpired:
+            process.kill()
+            console.print("🎪 ❌ Docker build timed out after 60 minutes")
+            return False
+
+        if return_code == 0:
             console.print(f"🎪 ✅ Docker build succeeded: {tag}")
             return True
         else:
-            console.print("🎪 ❌ Docker build failed:")
-            console.print(f"Exit code: {result.returncode}")
-            console.print(f"STDOUT: {result.stdout}")
-            console.print(f"STDERR: {result.stderr}")
+            console.print(f"🎪 ❌ Docker build failed with exit code: {return_code}")
             return False
-
-    except subprocess.TimeoutExpired:
-        console.print("🎪 ❌ Docker build timed out after 30 minutes")
-        return False
     except Exception as e:
         console.print(f"🎪 ❌ Docker build error: {e}")
         return False

{superset_showtime-0.2.9 → superset_showtime-0.3.1}/workflows-reference/showtime-trigger.yml

@@ -77,6 +77,13 @@ jobs:
           echo "pr_number=$PR_NUM_OUT" >> $GITHUB_OUTPUT
           echo "target_sha=$TARGET_SHA" >> $GITHUB_OUTPUT
 
+      - name: Checkout PR code (only if build needed)
+        if: steps.check.outputs.build_needed == 'true'
+        uses: actions/checkout@v4
+        with:
+          ref: ${{ steps.check.outputs.target_sha }}
+          persist-credentials: false
+
       - name: Setup Docker Environment (only if build needed)
         if: steps.check.outputs.build_needed == 'true'
         uses: ./.github/actions/setup-docker
@@ -85,13 +92,6 @@ jobs:
           dockerhub-token: ${{ env.DOCKERHUB_TOKEN }}
           build: "true"
 
-      - name: Checkout PR code (only if build needed)
-        if: steps.check.outputs.build_needed == 'true'
-        uses: actions/checkout@v4
-        with:
-          ref: ${{ steps.check.outputs.target_sha }}
-          persist-credentials: false
-
       - name: Execute sync (handles everything)
         if: steps.check.outputs.sync_needed == 'true'
         run: |