-
Notifications
You must be signed in to change notification settings - Fork 182
Closed
Labels
CI/CDCI/CD infrastructureCI/CD infrastructureP0High priority - Must do!High priority - Must do!enhancementAny code-related improvementsAny code-related improvements
Milestone
Description
We took the shortcut and used a 3rd-party action on the Windows test runners due to some technical reasons (the build runners already use fetch_ctk
just fine)
cuda-python/.github/workflows/test-wheel-windows.yml
Lines 161 to 165 in bd770e1
- name: Set up mini CTK | |
if: ${{ inputs.local-ctk == '1' }} | |
# Note: The GH-hosted Windows GPU runner does not have Git for Windows pre-installed, | |
# so we cannot use our own fetch_ctk action unfortunately... | |
uses: Jimver/cuda-toolkit@v0.2.21 |
It works very well most of the time, except when updating to the latest CTK release as evidenced in #597:
https://github.com/NVIDIA/cuda-python/actions/runs/14795896391/job/41543758103?pr=597#step:15:27
Because of this and other reasons (ex: #588), we probably should continue finding a way to enable
fetch_ctk
universally.Metadata
Metadata
Assignees
Labels
CI/CDCI/CD infrastructureCI/CD infrastructureP0High priority - Must do!High priority - Must do!enhancementAny code-related improvementsAny code-related improvements