Skip to content

Commit cfea985

Browse files
committed
Execute delete-pool always if nondocchanges == true (#6626)
## Motivation The current systest workflow fails when merging a change that doesn't execute system tests, i.e. only documentation was changed. This should fix the issue.
1 parent 35c94cd commit cfea985

File tree

1 file changed

+3
-2
lines changed

1 file changed

+3
-2
lines changed

.github/workflows/systest.yml

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -42,7 +42,7 @@ jobs:
4242
filter-changes:
4343
runs-on: ubuntu-22.04
4444
outputs:
45-
nondocchanges: ${{ steps.filter.outputs.nondoc }}
45+
nondocchanges: ${{ steps.filter.outputs.nondoc || github.event_name == 'pull_request' }}
4646
steps:
4747
- uses: actions/checkout@v4
4848
- uses: dorny/paths-filter@v3
@@ -172,6 +172,7 @@ jobs:
172172
systest-gke:
173173
runs-on: ubuntu-22.04
174174
if: ${{ needs.filter-changes.outputs.nondocchanges == 'true' }}
175+
continue-on-error: true
175176
needs:
176177
- filter-changes
177178
- provision-cluster
@@ -260,7 +261,7 @@ jobs:
260261

261262
delete-pool:
262263
runs-on: ubuntu-22.04
263-
if: always()
264+
if: ${{ needs.filter-changes.outputs.nondocchanges == 'true' }}
264265
needs:
265266
- filter-changes
266267
- systest-gke

0 commit comments

Comments
 (0)