Skip to content

Commit 4aa2874

Browse files
authored
Merge pull request #393 from leofang/cut_ci
Minor CI improvements
2 parents 291ed64 + acd5daf commit 4aa2874

File tree

2 files changed

+12
-6
lines changed

2 files changed

+12
-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') ||
@@ -143,7 +142,7 @@ jobs:
143142
PARALLEL_LEVEL=${{ env.PARALLEL_LEVEL }}
144143
CIBW_ENVIRONMENT_WINDOWS: >
145144
CUDA_HOME="$(cygpath -w ${{ env.CUDA_PATH }})"
146-
# PARALLEL_LEVEL=${{ env.PARALLEL_LEVEL }}
145+
PARALLEL_LEVEL=${{ env.PARALLEL_LEVEL }}
147146
with:
148147
package-dir: ./cuda_bindings/
149148
output-dir: ${{ env.CUDA_BINDINGS_ARTIFACTS_DIR }}
@@ -230,14 +229,18 @@ jobs:
230229
- 0 # use CTK wheels
231230
runner:
232231
- default
232+
exclude:
233+
# To test this combo would require nontrivial installation steps.
234+
- cuda-version: "12.0.1"
235+
local-ctk: 0
233236
include:
234237
- host-platform: linux-64
235238
python-version: "3.12"
236239
cuda-version: "12.6.2"
240+
local-ctk: 1
237241
runner: H100
238242
name: Test (${{ matrix.host-platform }}, Python ${{ matrix.python-version }}, CUDA ${{ matrix.cuda-version }}, Runner ${{ matrix.runner }}, ${{ (matrix.local-ctk == '1' && 'local CTK') || 'CTK wheels' }})
239-
# The build stage could fail but we want the CI to keep moving.
240-
if: ${{ github.repository_owner == 'nvidia' && !cancelled() }}
243+
if: ${{ github.repository_owner == 'nvidia' }}
241244
permissions:
242245
contents: read # This is required for actions/checkout
243246
needs:
@@ -255,8 +258,7 @@ jobs:
255258

256259
doc:
257260
name: Docs
258-
# The build stage could fail but we want the CI to keep moving.
259-
if: ${{ github.repository_owner == 'nvidia' && !cancelled() }}
261+
if: ${{ github.repository_owner == 'nvidia' }}
260262
# Sets permissions of the GITHUB_TOKEN to allow deployment to GitHub Pages
261263
permissions:
262264
id-token: write

.github/workflows/test-wheel.yml

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -122,13 +122,17 @@ jobs:
122122
echo "LATEST_PRIOR_RUN_ID not found!"
123123
exit 1
124124
fi
125+
125126
gh run download $LATEST_PRIOR_RUN_ID -p ${OLD_BASENAME} -R NVIDIA/cuda-python
126127
ls -al $OLD_BASENAME
127128
mkdir -p "${{ env.CUDA_BINDINGS_ARTIFACTS_DIR }}"
128129
mv $OLD_BASENAME/*.whl "${{ env.CUDA_BINDINGS_ARTIFACTS_DIR }}"/
130+
rmdir $OLD_BASENAME
129131
130132
gh run download $LATEST_PRIOR_RUN_ID -p cuda-python-wheel -R NVIDIA/cuda-python
133+
ls -al cuda-python-wheel
131134
mv cuda-python-wheel/*.whl .
135+
rmdir cuda-python-wheel
132136
133137
- name: Display structure of downloaded cuda-python artifacts
134138
run: |

0 commit comments

Comments
 (0)