[Tool] Ignore result upload if UT is failed (#26878)

Signed-off-by: AndyZiYe <[email protected]>
StarRocks · Jul 10, 2023 · 5ce14b4 · 5ce14b4
1 parent 5e0eb6b
commit 5ce14b4
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 2 deletions.
diff --git a/.github/workflows/ci-merged.yml b/.github/workflows/ci-merged.yml
@@ -126,6 +126,8 @@ jobs:
           echo "commit_sha=${commit_sha}" >> $GITHUB_OUTPUT
 
       - name: update merged coverage
+        env:
+          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
         run: |
           rm -rf ./ci-tool && cp -rf /var/lib/ci-tool ./ci-tool && cd ci-tool && git pull && source lib/init.sh
           ./bin/upload_cov_after_merge.sh --pr ${PR_NUMBER} --commit ${{ steps.commit_sha.outputs.commit_sha }}

diff --git a/.github/workflows/ci-pipeline-branch.yml b/.github/workflows/ci-pipeline-branch.yml
@@ -430,7 +430,7 @@ jobs:
           verbose: true
 
       - name: Upload Result
-        if: always()
+        if: steps.run_ut.outcome == 'success'
         run: |
           branch=${{ steps.branch.outputs.branch }}
           bucket_prefix=${{ steps.branch.outputs.bucket_prefix }}

diff --git a/.github/workflows/ci-pipeline.yml b/.github/workflows/ci-pipeline.yml
@@ -475,7 +475,7 @@ jobs:
           verbose: true
 
       - name: Upload Result
-        if: always()
+        if: steps.run_ut.outcome == 'success'
         run: |
           branch=${{ steps.branch.outputs.branch }}
           bucket_prefix=${{ steps.branch.outputs.bucket_prefix }}