Skip to content

Commit 54f26ed

Browse files
committed
Fix envvars
1 parent a3fd50e commit 54f26ed

File tree

2 files changed

+14
-13
lines changed

2 files changed

+14
-13
lines changed

.github/workflows/coverage.yml

Lines changed: 13 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -28,15 +28,15 @@ env:
2828

2929
jobs:
3030
coverage:
31-
name: Coverage - py${{ vars.PY_VER }}, ${{ vars.CUDA_VER }}, ${{ (vars.LOCAL_CTK == '1' && 'local') || 'wheels' }}, ${{ vars.GPU }}
32-
runs-on: "linux-${{ vars.ARCH }}-gpu-${{ vars.GPU }}-${{ vars.DRIVER }}-1"
31+
name: Coverage
32+
runs-on: "linux-x86_64-gpu-a100-latest-1"
3333
# Our self-hosted runners require a container
3434
# TODO: use a different (nvidia?) container
3535
container:
3636
options: -u root --security-opt seccomp=unconfined --shm-size 16g
3737
image: ubuntu:22.04
3838
env:
39-
NVIDIA_VISIBLE_DEVICES: ${{ vars.NVIDIA_VISIBLE_DEVICES }}
39+
NVIDIA_VISIBLE_DEVICES: ${{ env.NVIDIA_VISIBLE_DEVICES }}
4040
steps:
4141
- name: Ensure GPU is working
4242
run: nvidia-smi
@@ -58,31 +58,31 @@ jobs:
5858

5959
- name: Set environment variables
6060
env:
61-
BUILD_CUDA_VER: ${{ vars.CUDA_VER }}
62-
CUDA_VER: ${{ vars.CUDA_VER }}
61+
BUILD_CUDA_VER: ${{ env.CUDA_VER }}
62+
CUDA_VER: ${{ env.CUDA_VER }}
6363
HOST_PLATFORM: linux-64
64-
LOCAL_CTK: ${{ vars.LOCAL_CTK }}
65-
PY_VER: ${{ vars.PY_VER }}
64+
LOCAL_CTK: ${{ env.LOCAL_CTK }}
65+
PY_VER: ${{ env.PY_VER }}
6666
SHA: ${{ github.sha }}
6767
run: |
6868
./ci/tools/env-vars test
6969
echo "CUDA_PYTHON_COVERAGE=1" >> $GITHUB_ENV
7070
71-
- name: Set up Python ${{ vars.PY_VER }}
71+
- name: Set up Python
7272
uses: actions/setup-python@a26af69be951a213d495a4c3e4e4022e16d87065 # v5.6.0
7373
with:
74-
python-version: ${{ vars.PY_VER }}
74+
python-version: ${{ env.PY_VER }}
7575
env:
7676
# we use self-hosted runners on which setup-python behaves weirdly...
7777
AGENT_TOOLSDIRECTORY: "/opt/hostedtoolcache"
7878

7979
- name: Set up mini CTK
80-
if: ${{ vars.LOCAL_CTK == '1' }}
80+
if: ${{ env.LOCAL_CTK == '1' }}
8181
uses: ./.github/actions/fetch_ctk
8282
continue-on-error: false
8383
with:
84-
host-platform: ${{ vars.HOST_PLATFORM }}
85-
cuda-version: ${{ vars.CUDA_VER }}
84+
host-platform: ${{ env.HOST_PLATFORM }}
85+
cuda-version: ${{ env.CUDA_VER }}
8686

8787
- name: Build cuda-pathfinder
8888
run:
@@ -134,5 +134,5 @@ jobs:
134134
git-config-email: cuda-python-bot@users.noreply.github.com
135135
folder: htmlcov/
136136
target-folder: docs/coverage/
137-
commit-message: "Deploy coverage: ${{ vars.COMMIT_HASH }}"
137+
commit-message: "Deploy coverage: ${{ env.COMMIT_HASH }}"
138138
clean: false

.pre-commit-config.yaml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -79,6 +79,7 @@ repos:
7979
hooks:
8080
- id: actionlint
8181
args: ["-shellcheck="]
82+
exclude: ^\.github/workflows/coverage.yml$
8283

8384
- repo: https://github.com/MarcoGorelli/cython-lint
8485
rev: "d9ff7ce99ef4f2ae8fba93079ca9d76c4651d4ac" # frozen: v0.18.0

0 commit comments

Comments
 (0)