Skip to content

Commit fd93332

Browse files
Cleanup CI
Signed-off-by: Keval Morabia <[email protected]>
1 parent add6912 commit fd93332

File tree

6 files changed

+111
-81
lines changed

6 files changed

+111
-81
lines changed

.github/codecov.yml

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,8 @@
1+
# Allow atmost 5% coverage drop from main branch.
2+
coverage:
3+
status:
4+
project:
5+
default:
6+
target: auto
7+
threshold: 5%
8+
patch: false
Lines changed: 29 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,29 @@
1+
name: Wait for checks
2+
3+
on:
4+
workflow_call:
5+
inputs:
6+
match_pattern:
7+
required: true
8+
type: string
9+
delay:
10+
required: false
11+
type: string
12+
default: 10s
13+
14+
jobs:
15+
wait:
16+
runs-on: ubuntu-latest
17+
permissions:
18+
checks: read
19+
steps:
20+
- name: Wait for checks (PRs only)
21+
if: github.event_name == 'pull_request' || startsWith(github.ref, 'refs/heads/pull-request/')
22+
uses: poseidon/[email protected]
23+
with:
24+
token: ${{ secrets.GITHUB_TOKEN }}
25+
match_pattern: ${{ inputs.match_pattern }}
26+
delay: ${{ inputs.delay }}
27+
- name: No-op for non-PR events
28+
if: github.event_name != 'pull_request' && !startsWith(github.ref, 'refs/heads/pull-request/')
29+
run: echo "Not a pull_request event"

.github/workflows/gpu_tests.yml

Lines changed: 18 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,11 +14,28 @@ on:
1414

1515
# Cancel previous runs if new commit is pushed to the same PR
1616
concurrency:
17-
group: ${{ github.workflow }}-${{ github.event.pull_request.number }}
17+
group: ${{ github.workflow }}-${{ github.ref }}
1818
cancel-in-progress: true
1919

2020
jobs:
21+
wait-dco:
22+
uses: ./.github/workflows/_wait_for_checks.yml
23+
permissions:
24+
checks: read
25+
secrets: inherit
26+
with:
27+
match_pattern: '^DCO$'
28+
wait-unit-tests:
29+
needs: [wait-dco]
30+
uses: ./.github/workflows/_wait_for_checks.yml
31+
permissions:
32+
checks: read
33+
secrets: inherit
34+
with:
35+
match_pattern: '^Unit tests / linux$'
36+
delay: 60s
2137
gpu-tests:
38+
needs: [wait-unit-tests]
2239
# Runner list at https://github.com/nv-gha-runners/enterprise-runner-configuration/blob/main/docs/runner-groups.md
2340
runs-on: linux-amd64-gpu-h100-latest-1
2441
timeout-minutes: 60

.github/workflows/multi_version_unit_tests.yml

Lines changed: 0 additions & 46 deletions
This file was deleted.

.github/workflows/partial_unit_tests.yml

Lines changed: 0 additions & 32 deletions
This file was deleted.

.github/workflows/unit_tests.yml

Lines changed: 56 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,15 @@ concurrency:
2626
cancel-in-progress: true
2727

2828
jobs:
29-
unit-tests:
29+
wait-dco:
30+
uses: ./.github/workflows/_wait_for_checks.yml
31+
permissions:
32+
checks: read
33+
secrets: inherit
34+
with:
35+
match_pattern: '^DCO$'
36+
linux:
37+
needs: [wait-dco]
3038
runs-on: ubuntu-latest
3139
timeout-minutes: 30
3240
steps:
@@ -42,8 +50,9 @@ jobs:
4250
token: ${{ secrets.CODECOV_TOKEN }}
4351
fail_ci_if_error: true
4452
verbose: true
45-
unit-tests-windows:
53+
windows:
4654
if: github.event_name == 'pull_request'
55+
needs: [linux]
4756
runs-on: windows-latest
4857
timeout-minutes: 30
4958
steps:
@@ -53,3 +62,48 @@ jobs:
5362
python-version: "3.12"
5463
- name: Run unit tests (without coverage)
5564
run: pip install tox && tox -e py312-torch28-unit
65+
multi-py:
66+
if: github.event_name == 'pull_request'
67+
needs: [linux]
68+
runs-on: ubuntu-latest
69+
timeout-minutes: 30
70+
strategy:
71+
matrix:
72+
py: [10, 11]
73+
steps:
74+
- uses: actions/checkout@v4
75+
- uses: actions/setup-python@v5
76+
with:
77+
python-version: "3.${{ matrix.py }}"
78+
- name: Run unit tests
79+
run: pip install tox && tox -e py3${{ matrix.py }}-torch28-unit
80+
multi-torch:
81+
if: github.event_name == 'pull_request'
82+
needs: [linux]
83+
runs-on: ubuntu-latest
84+
timeout-minutes: 30
85+
strategy:
86+
matrix:
87+
torch: [25, 26, 27]
88+
steps:
89+
- uses: actions/checkout@v4
90+
- uses: actions/setup-python@v5
91+
with:
92+
python-version: "3.12"
93+
- name: Run unit tests
94+
run: pip install tox && tox -e py312-torch${{ matrix.torch }}-unit
95+
partial-install:
96+
if: github.event_name == 'pull_request'
97+
needs: [linux]
98+
runs-on: ubuntu-latest
99+
timeout-minutes: 30
100+
strategy:
101+
matrix:
102+
test-env: [onnx, torch]
103+
steps:
104+
- uses: actions/checkout@v4
105+
- uses: actions/setup-python@v5
106+
with:
107+
python-version: "3.12"
108+
- name: Run unit tests
109+
run: pip install tox && tox -e py312-partial-unit-${{ matrix.test-env }}

0 commit comments

Comments
 (0)