Skip to content

Commit

Permalink
run systest-status CI job on PR (#6471)
Browse files Browse the repository at this point in the history
  • Loading branch information
poszu authored Nov 19, 2024
1 parent 6e84799 commit d6ddd0b
Showing 1 changed file with 5 additions and 3 deletions.
8 changes: 5 additions & 3 deletions .github/workflows/systest.yml
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,7 @@ on:
description: "Log level"
default: "debug"
merge_group:
pull_request:

env:
GCLOUD_KEY: ${{ secrets.GCLOUD_KEY }}
Expand Down Expand Up @@ -52,7 +53,7 @@ jobs:
systest:
runs-on: ubuntu-22.04
if: ${{ needs.filter-changes.outputs.nondocchanges == 'true' }}
if: ${{ needs.filter-changes.outputs.nondocchanges == 'true' && github.event_name != 'pull_request' }}
needs:
- filter-changes
timeout-minutes: 70
Expand Down Expand Up @@ -157,8 +158,9 @@ jobs:
- systest
runs-on: ubuntu-22.04
env:
# short-circuit success if no non-doc files were modified
status: ${{ (needs.filter-changes.outputs.nondocchanges == 'false' || needs.systest.result == 'success') && 'success' || 'failure' }}
# On pull_request: always success
# On merge_group: success if no non-doc changes or if systest passed
status: ${{ github.event_name == 'pull_request' && 'success' || (needs.filter-changes.outputs.nondocchanges == 'false' || needs.systest.result == 'success') && 'success' || 'failure' }}
steps:
# print a single, clean status update to slack
- uses: act10ns/slack@v2
Expand Down

0 comments on commit d6ddd0b

Please sign in to comment.