Skip to content

Commit a888a83

Browse files
Add last aggregator job in unit and gpu test to use as required check in PRs
Signed-off-by: Keval Morabia <[email protected]>
1 parent 2b52759 commit a888a83

File tree

2 files changed

+15
-0
lines changed

2 files changed

+15
-0
lines changed

.github/workflows/gpu_tests.yml

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -70,3 +70,12 @@ jobs:
7070
timeout-minutes: 90
7171
container: *gpu_container
7272
steps: *gpu_steps
73+
gpu-pr-required-check:
74+
# Run even if gpu-tests-pr is skipped
75+
if: ${{ startsWith(github.ref, 'refs/heads/pull-request/') && always() }}
76+
needs: [check-file-changes, gpu-tests-pr]
77+
runs-on: ubuntu-latest
78+
steps:
79+
- name: Required GPU tests did not succeed
80+
if: ${{ needs.check-file-changes.outputs.any_changed == 'true' && needs.gpu-tests-pr.result != 'success' }}
81+
run: exit 1

.github/workflows/unit_tests.yml

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -126,3 +126,9 @@ jobs:
126126
python-version: "3.12"
127127
- name: Run unit tests
128128
run: pip install tox && tox -e py312-partial-unit-${{ matrix.test-env }}
129+
unit-pr-required-check:
130+
if: github.event_name == 'pull_request'
131+
needs: [linux, windows, multi-py, multi-torch, multi-transformers, partial-install]
132+
runs-on: ubuntu-latest
133+
steps:
134+
- run: echo "All PR unit test jobs completed"

0 commit comments

Comments
 (0)