mirror of
https://github.com/apache/superset.git
synced 2026-05-25 01:35:39 +00:00
fix: Match workflow conditions with actual showtime CLI output
Update output extraction to use 'sync_needed' and 'target_sha' to match what showtime CLI actually returns. This fixes the issue where sync steps were skipped despite sync_needed=true. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
This commit is contained in:
12
.github/workflows/showtime-trigger.yml
vendored
12
.github/workflows/showtime-trigger.yml
vendored
@@ -79,15 +79,15 @@ jobs:
|
||||
OUTPUT=$(python -m showtime sync $PR_NUM --check-only)
|
||||
echo "$OUTPUT"
|
||||
|
||||
# Extract outputs for conditional steps
|
||||
ACTION=$(echo "$OUTPUT" | grep "action_needed=" | cut -d'=' -f2)
|
||||
# Extract outputs for conditional steps (match showtime CLI output)
|
||||
BUILD=$(echo "$OUTPUT" | grep "build_needed=" | cut -d'=' -f2)
|
||||
DEPLOY=$(echo "$OUTPUT" | grep "deploy_needed=" | cut -d'=' -f2)
|
||||
SYNC=$(echo "$OUTPUT" | grep "sync_needed=" | cut -d'=' -f2)
|
||||
TARGET_SHA=$(echo "$OUTPUT" | grep "target_sha=" | cut -d'=' -f2)
|
||||
|
||||
echo "action_needed=$ACTION" >> $GITHUB_OUTPUT
|
||||
echo "build_needed=$BUILD" >> $GITHUB_OUTPUT
|
||||
echo "deploy_needed=$DEPLOY" >> $GITHUB_OUTPUT
|
||||
echo "sync_needed=$SYNC" >> $GITHUB_OUTPUT
|
||||
echo "pr_number=$PR_NUM" >> $GITHUB_OUTPUT
|
||||
echo "target_sha=$TARGET_SHA" >> $GITHUB_OUTPUT
|
||||
|
||||
- name: Get SHA for potential build
|
||||
id: sha
|
||||
@@ -172,7 +172,7 @@ jobs:
|
||||
echo "docker_tag=$CUSTOM_TAG" >> $GITHUB_OUTPUT
|
||||
|
||||
- name: Execute sync deployment
|
||||
if: steps.check.outputs.deploy_needed == 'true'
|
||||
if: steps.check.outputs.sync_needed == 'true'
|
||||
run: |
|
||||
PR_NUM="${{ steps.check.outputs.pr_number }}"
|
||||
|
||||
|
||||
Reference in New Issue
Block a user