You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
[no-ci] ci: build and test local CUDA 12 bindings - #3317
Layer 3 of 7 of the staged "CUDA 12 bindings on main" stack. Previous: #3315. Next: #3318. Stack map at the bottom.
Merge sequencing: land the five remaining migration PRs together using the atomic stack merge described in the landing plan.
What this PR does
main starts building the CUDA 12 bindings wheel itself and running its GPU test suite, next to the existing CUDA 13 pipeline. Nothing else moves yet: cuda.core and the metapackage keep consuming the 12.9.x-branch artifacts (that cord is cut in #3318).
build-wheel.yml: after the CUDA 13 work, switch to the maintenance mini-CTK (already fetched for Core's cu12 half), build the cuda_bindings_12 wheel with the same cibuildwheel environment (shared through a YAML anchor), twine check it, upload it, and build and upload its Cython test extensions.
ci.yml: three new jobs, Maintenance bindings (linux-64 / linux-aarch64 / win-64), call the existing test workflows with maintenance-bindings: true. Only matrix rows with CUDA_VER == 12.9.1 run, and only the bindings suite.
test-wheel-linux.yml, test-wheel-windows.yml: new inputs bindings-config and maintenance-bindings; rows gain BINDINGS_ROOT.
ci/tools/env-vars, ci/tools/run-tests: CUDA_BINDINGS_ROOT replaces the hard-coded cuda_bindings_13.
cuda_bindings_12/pyproject.toml: wheel payload policy copied from CUDA 13 (include-package-data = false: wheels ship .pxd/.pxi and headers, not .pyx sources or .in templates; sdists keep everything) and cibuildwheel settings. Recorded in the upcoming release notes.
Temporary scaffolding: the artifact name maintenance-bindings-python<py>-cuda12.9.1-<platform>-<sha> and the three jobs exist only in this layer. #3318 folds the tests into the regular jobs and #3319 renames the artifacts to the final cuda-bindings-python<py>-cuda<toolkit>-... scheme.
After this merges
Every CI run additionally builds one CUDA 12 bindings wheel per Python version and platform (no Windows ARM64; CUDA 12 has no toolkit there) and runs the bindings suite on the matrix rows whose toolkit is 12.9.1 (a few extra GPU jobs per platform).
cuda.core, the metapackage, releases, and nightly are unchanged.
ci/tools/run-tests: bindings suite runs from CUDA_BINDINGS_ROOT.
cuda_bindings_12/pyproject.toml, docs/source/release/12.X.Y-notes.rst: payload policy and its note.
Validation
Full CI passed at the pre-rebase head afe9fc9e (run 41185002774), including the three new maintenance-bindings jobs.
Local validation at 9794e83edb7875a53849f562a17043cf79d88797: 316 CI-tool tests and configured workflow lint passed. Full-stack CI on #3322 passed at 126ed74fb5bbfbc91a04e50557cf936efef4da7f (105 successful jobs; one expected skip).
adopts the schema-2 registry, removes backport automation, adds metadata checks and maintenance docs; 12.9.x is frozen
Layers #3314 and #3315 have merged. The remaining five migration PRs (#3317, #3318, #3319, #3321 and #3322) will land together using GitHub’s atomic stack merge, with full CI on #3322 as described in the landing plan. #3434 follows separately, after a successful post-migration main CI run supplies complete artifacts for both CUDA majors.
The reason will be displayed to describe this comment to others. Learn more.
This PR is a bit of a special case to call it out. It adds things that we wouldn't want permanently that subsequence steps remove, really to just "bootstrap" the wheels into the right place. It would be better if there were a way to bootstrap in a single step because this will put us on a path to merge each of these individually.
I think a better arrangement would be to have a lower-level wheel-building workflow for specific wheels that could be triggered manually. Then for bootstrapping, we kick it off directly, and otherwise it would just be used in various configurations from the existing build-wheel.yml script. But that's probably not worth it.
🔴 JUST AN IMPORTANT NOTE TO MAKE SURE THE STACK ABOVE THIS IS APPROVED BEFORE MERGING THIS SO WE DON'T GET STUCK IN THIS BAD INTERMEDIATE STATE.
The reason will be displayed to describe this comment to others. Learn more.
— codex GPT-6 on behalf of Ralf —
This caller is now removed as part of the common bootstrap cleanup. The retained v12.9.11a0 milestone is on main and both alternatives are restacked onto it, as recorded in the updated #3315 reply.
The reason will be displayed to describe this comment to others. Learn more.
— codex GPT-6 on behalf of Ralf —
Thanks for calling out the intermediate-state risk in your review. The PR description now explicitly requires approval of the upper stack before this temporary bridge merges. I kept the existing bridge rather than introducing another wheel-building workflow.
All three inline comments now have direct replies. The Python/input-name changes are applied. The retained v12.9.11a0 milestone is now on main; both alternatives are restacked onto it, and the bootstrap helper, tests, metadata and callers are removed.
The reason will be displayed to describe this comment to others. Learn more.
Minor nit: We don't need both x86 and arm for windows on the maintenance build today, but when we move from 12/13 to 13/14 we will need to add Windows on ARM here, so maybe add a comment?
Base automatically changed from
rwgk/cb12-staged-02-developer to
mainNovember 6, 2026 19:28
No actionable comments were generated in the recent review.
Summary
Summary by CodeRabbit
New Features
Added maintenance-version CUDA bindings wheel builds and automated tests for supported Linux and Windows platforms.
Documentation
Clarified that wheels include Cython declarations and required headers but omit implementation sources and generation templates; source distributions retain the inputs needed to build the package.
Chores
Updated wheel packaging settings, including Windows wheel repair and Linux build compatibility, and improved caching for bindings builds.
The CI workflows now build prior-major maintenance cuda.bindings wheels and run bindings tests on Linux and Windows. The CUDA 12 package configuration defines wheel contents and build settings. Release notes describe the packaged files.
Changes: Maintenance cuda.bindings pipeline
Layer / File(s)
Summary
CUDA 12 wheel packaging cuda_bindings_12/pyproject.toml, cuda_bindings_12/docs/source/release/12.X.Y-notes.rst
The package configuration specifies included Cython declarations and headers, wheel platforms, and Windows repair. The release notes describe wheel and source distribution contents.
Maintenance wheel build and Cython artifacts .github/workflows/build-wheel.yml, ci/tools/env-vars, ci/tools/run-tests
When prior-major builds are enabled, the workflow builds and checks a maintenance wheel for bindings builds or tests. For bindings tests, it installs that wheel, builds Cython tests, and uploads the artifacts. The environment scripts resolve maintenance package paths and artifact names.
Maintenance test matrix and CI orchestration .github/workflows/ci.yml, .github/workflows/test-wheel-linux.yml, .github/workflows/test-wheel-windows.yml, ci/tools/env-vars, ci/tools/run-tests
Maintenance mode selects package roots by toolkit version and runs bindings tests only. The CI workflow schedules maintenance test jobs for Linux x86-64, Linux AArch64, and Windows, then checks their expected results.
Possibly related issues
NVIDIA/cuda-python#3402 — The issue asks for a shared Core wheel-build implementation. This PR adds maintenance bindings wheel steps to the same workflow but does not extract that implementation.
Possibly related PRs
NVIDIA/cuda-python#3321 — It changes the build and bindings routing contracts used by this PR’s maintenance workflow.
NVIDIA/cuda-python#3318 — It changes maintenance-bindings inputs, build conditions, and environment variables also changed here.
NVIDIA/cuda-python#3322 — It revises the registry and artifact paths used by this maintenance flow.
Suggested reviewers:leofang, jpascucci-nv
Change: Other
Merge Risk: Minimal · up to d179c
This change adds CUDA 12 bindings wheel builds and tests to CI alongside the CUDA 13 pipeline. No concrete merge-blocking defect was found in the supplied review context.
Architecture risk: Low · up to d179c
The change affects 2 systems.
Changed systems:ci, cuda_bindings_12
Architecture concerns
No architecture-level concerns identified.
Review details
Systems and components
observed — ci (service) was modified; 2 changed files map to changed impact.
observed — cuda_bindings_12 (service) was modified; 2 changed files map to changed impact.
Before / after behavior
observed — Modified behavior in ci/tools/env-vars: Adds defaults for the bindings package root and artifact-name prefix; existing environment values take precedence.
observed — Modified behavior in ci/tools/env-vars: Build mode queries ci.tools.bindings_config for the maintenance release’s package root and toolkit version, then exports those values, resolved distribution and Cython-test paths, and an artifact name containing the Python version, toolkit version, platform, and SHA.
observed — Modified behavior in ci/tools/env-vars: The test-mode bindings artifact basename now uses BINDINGS_ARTIFACT_PREFIX instead of the fixed cuda-bindings prefix.
observed — Modified behavior in ci/tools/env-vars: The exported bindings artifact and Cython-test directories now resolve beneath CUDA_BINDINGS_ROOT instead of the fixed cuda_bindings_13 path.
- [ ] Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
Reviewing files that changed from the base of the PR and between c707f8f and d179cd9.
Files selected for processing: 1
.github/workflows/build-wheel.yml
Included review availability: This review used your included allowance. Your plan provides up to 12 included reviews per hour; 11 remain after this review.
The reason will be displayed to describe this comment to others. Learn more.
Additional comments: 1
.github/workflows/build-wheel.yml: 1
563-563: Trivial · Maintainability & Code Quality · Low value
Unverified finding
Verification ran but could not confirm this finding. It is shown for review, not as a verified issue.
suggestion: This step pins pypa/cibuildwheel at v4.3.4 (b9e566a2). The other cibuildwheel steps in this file pin v4.3.5 (f9b966cf). Use the same pin for all steps so the CUDA 12 and CUDA 13 wheels build with one tool version. If the older pin is intentional, add a comment that says why.
Included review availability: This review used your included allowance. Your plan provides up to 12 included reviews per hour; 8 remain after this review.
Addressed the cibuildwheel pin mismatch from this review in this update. The existing v4.3.5 alignment now belongs to this layer, so the CUDA 12 and CUDA 13 builders use the same version throughout the remaining stack.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
CI/CDCI/CD infrastructurecuda.bindingsEverything related to the cuda.bindings moduleenhancementAny code-related improvements
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
— codex GPT-6 on behalf of Ralf —
Layer 3 of 7 of the staged "CUDA 12 bindings on
main" stack. Previous: #3315. Next: #3318. Stack map at the bottom.Merge sequencing: land the five remaining migration PRs together using the atomic stack merge described in the landing plan.
What this PR does
mainstarts building the CUDA 12 bindings wheel itself and running its GPU test suite, next to the existing CUDA 13 pipeline. Nothing else moves yet:cuda.coreand the metapackage keep consuming the12.9.x-branch artifacts (that cord is cut in #3318).build-wheel.yml: after the CUDA 13 work, switch to the maintenance mini-CTK (already fetched for Core's cu12 half), build thecuda_bindings_12wheel with the same cibuildwheel environment (shared through a YAML anchor),twine checkit, upload it, and build and upload its Cython test extensions.ci.yml: three new jobs,Maintenance bindings (linux-64 / linux-aarch64 / win-64), call the existing test workflows withmaintenance-bindings: true. Only matrix rows withCUDA_VER == 12.9.1run, and only the bindings suite.test-wheel-linux.yml,test-wheel-windows.yml: new inputsbindings-configandmaintenance-bindings; rows gainBINDINGS_ROOT.ci/tools/env-vars,ci/tools/run-tests:CUDA_BINDINGS_ROOTreplaces the hard-codedcuda_bindings_13.cuda_bindings_12/pyproject.toml: wheel payload policy copied from CUDA 13 (include-package-data = false: wheels ship.pxd/.pxiand headers, not.pyxsources or.intemplates; sdists keep everything) and cibuildwheel settings. Recorded in the upcoming release notes.Temporary scaffolding: the artifact name
maintenance-bindings-python<py>-cuda12.9.1-<platform>-<sha>and the three jobs exist only in this layer. #3318 folds the tests into the regular jobs and #3319 renames the artifacts to the finalcuda-bindings-python<py>-cuda<toolkit>-...scheme.After this merges
cuda.core, the metapackage, releases, and nightly are unchanged.Where to look
.github/workflows/build-wheel.yml: the "maintenance cuda.bindings" steps near the bottom and the&bindings-wheel-envanchor..github/workflows/test-wheel-linux.yml: themaintenance-bindingsmatrix filter (the Windows file mirrors it).cuda_bindings_12/pyproject.toml: the payload policy, a deliberate product decision.All changes (8 files, +254 / -19)
.github/workflows/build-wheel.yml: maintenance wheel build/check/upload, Cython test build/upload, mini-CTK gating..github/workflows/ci.yml:ci-varsexports the registry JSON; threetest-maintenance-bindings-*jobs; thechecksaggregator expects them..github/workflows/test-wheel-linux.yml,test-wheel-windows.yml:bindings-config/maintenance-bindingsinputs, matrix filter, artifact prefix.ci/tools/env-vars:MAINTENANCE_*build variables,CUDA_BINDINGS_ROOT, artifact prefix.ci/tools/run-tests: bindings suite runs fromCUDA_BINDINGS_ROOT.cuda_bindings_12/pyproject.toml,docs/source/release/12.X.Y-notes.rst: payload policy and its note.Validation
afe9fc9e(run 41185002774), including the three new maintenance-bindings jobs.9794e83edb7875a53849f562a17043cf79d88797: 316 CI-tool tests and configured workflow lint passed. Full-stack CI on #3322 passed at126ed74fb5bbfbc91a04e50557cf936efef4da7f(105 successful jobs; one expected skip).Stack map: cumulative capabilities by layer
v12.*tagcuda_bindings_12/; the PR-time CPU source-build smoke covers both roots; three CUDA 12 runtime fixes are incuda.coreand the metapackage still consume12.9.xartifacts12.9.xbranchv12.9.Ntag onmain; CUDA 13 releases are unchanged12.9.xis frozenLayers #3314 and #3315 have merged. The remaining five migration PRs (#3317, #3318, #3319, #3321 and #3322) will land together using GitHub’s atomic stack merge, with full CI on #3322 as described in the landing plan. #3434 follows separately, after a successful post-migration
mainCI run supplies complete artifacts for both CUDA majors.