Skip to content

Commit a415b2e

Browse files
committed
exclude redundant test jobs; ensure build passes; enable win parallel build
1 parent c09e914 commit a415b2e

File tree

1 file changed

+8
-6
lines changed

1 file changed

+8
-6
lines changed

.github/workflows/build-and-test.yml

Lines changed: 8 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,6 @@ jobs:
3535
name: Build (${{ matrix.host-platform }}, Python ${{ matrix.python-version }}, CUDA ${{ matrix.cuda-version }})
3636
if: ${{ github.repository_owner == 'nvidia' }}
3737
permissions:
38-
id-token: write # This is required for configure-aws-credentials
3938
contents: read # This is required for actions/checkout
4039
runs-on: ${{ (matrix.host-platform == 'linux-64' && 'linux-amd64-cpu8') ||
4140
(matrix.host-platform == 'linux-aarch64' && 'linux-arm64-cpu8') ||
@@ -144,7 +143,7 @@ jobs:
144143
PARALLEL_LEVEL=${{ env.PARALLEL_LEVEL }}
145144
CIBW_ENVIRONMENT_WINDOWS: >
146145
CUDA_HOME="$(cygpath -w ${{ env.CUDA_PATH }})"
147-
# PARALLEL_LEVEL=${{ env.PARALLEL_LEVEL }}
146+
PARALLEL_LEVEL=${{ env.PARALLEL_LEVEL }}
148147
with:
149148
package-dir: ./cuda_bindings/
150149
output-dir: ${{ env.CUDA_BINDINGS_ARTIFACTS_DIR }}
@@ -203,14 +202,18 @@ jobs:
203202
- 0 # use CTK wheels
204203
runner:
205204
- default
205+
exclude:
206+
# To test this combo would require nontrivial installation steps.
207+
- cuda-version: "12.0.1"
208+
local-ctk: 0
206209
include:
207210
- host-platform: linux-64
208211
python-version: "3.12"
209212
cuda-version: "12.6.2"
213+
local-ctk: 1
210214
runner: H100
211215
name: Test (${{ matrix.host-platform }}, Python ${{ matrix.python-version }}, CUDA ${{ matrix.cuda-version }}, Runner ${{ matrix.runner }}, ${{ (matrix.local-ctk == '1' && 'local CTK') || 'CTK wheels' }})
212-
# The build stage could fail but we want the CI to keep moving.
213-
if: ${{ github.repository_owner == 'nvidia' && !cancelled() }}
216+
if: ${{ github.repository_owner == 'nvidia' }}
214217
permissions:
215218
contents: read # This is required for actions/checkout
216219
needs:
@@ -228,8 +231,7 @@ jobs:
228231

229232
doc:
230233
name: Docs
231-
# The build stage could fail but we want the CI to keep moving.
232-
if: ${{ github.repository_owner == 'nvidia' && !cancelled() }}
234+
if: ${{ github.repository_owner == 'nvidia' }}
233235
# Sets permissions of the GITHUB_TOKEN to allow deployment to GitHub Pages
234236
permissions:
235237
id-token: write

0 commit comments

Comments
 (0)