Commit 45c8b24
authored
CI: nightly-cuda-core + nightly-numba-cuda-mlir + PTDS fix (#2283)
* CI: add nightly-cuda-core and nightly-numba-cuda-mlir modes
nightly-cuda-core: test the released cuda-core from PyPI against
main-built pathfinder and cuda-bindings, catching the "core released ×
bindings main" gap documented in issue #1955. Runs on linux-64 (a100)
and win-64 (a100 MCDM).
nightly-numba-cuda-mlir: MLIR-backend companion to nightly-numba-cuda.
Installs main pathfinder+bindings+core plus numba-cuda-mlir from PyPI,
runs numba-cuda-mlir's own test suite from the matching git tag.
Linux amd64/arm64 x CUDA 12.9.1 / 13.3.0.
Both modes fetch the released version's tests from git tags because
the respective wheels do not ship test_*.py files. Includes
tag-not-found fallback (log warning + exit 0) to avoid red-lining the
nightly on a freshly-cut PyPI release that hasn't been pushed to git
yet.
* ci/test-matrix.yml: fix CUDA_PYTHON_CUDA_PER_THREAD_DEFAULT_STREAM typo
The two ENV overrides intended to exercise the per-thread default
stream code path were misspelled (missing the CUDA_ segment), so the
env var was silently ignored and the PTDS coverage added in #1972 had
no effect. Rename to the correct
CUDA_PYTHON_CUDA_PER_THREAD_DEFAULT_STREAM.
Refs #971.
* cuda_pathfinder: pin nvshmem to <3.7 (was previously excluding only 3.7.0)
nvidia-nvshmem-cu{12,13} 3.7.x breaks the main branch, not only 3.7.0. Widen the exclusion from an exact-version bump to <3.7 so 3.7.x and above are avoided until we can move forward.
* nightly-numba-cuda-mlir: swap arm64 for win-64 coverage, use rtxpro6000
Drop the linux-aarch64 rows and instead add win-64 coverage with the
same CUDA 12.9.1 / 13.3.0 pair. Switch all four rows from GPU l4 to
rtxpro6000. Windows rows use DRIVER_MODE MCDM, matching the existing
rtxpro6000 CUDA 13.3.0 patterns.
* Temporarily add push trigger to ci-nightly.yml for testing
Remove before merging.
* CI: switch nightly-{cuda-core,numba-cuda-mlir} to actions/checkout for tests
The initial approach used git inside the ubuntu:24.04 container to fetch
the released version's test suite, but git is not installed on that
container (install_unix_deps only pulls in jq/wget/g++/etc.) and its
absence made the run steps silently skip via the tag-not-fetchable
fallback. On Windows, git archive of just the cuda_core subtree also hit
a dangling-symlink extraction failure (cuda_core/.git_archival.txt).
Refactor to:
- run-tests: just install wheels and expose the resolved release version
(CUDA_CORE_RELEASED_VER / NUMBA_CUDA_MLIR_VER) and cuda-core test-group
name via GITHUB_ENV. No more git operations.
- test-wheel-{linux,windows}.yml: add an actions/checkout step per mode
that pulls the matching release tag into a subdirectory
(cuda-core-released / numba-cuda-mlir-released), then the follow-up
test step installs that tag's test dep-group and runs pytest.
For numba-cuda-mlir also pass --ignore=tests/benchmarks
--ignore=tests/doc_examples to pytest: those directories import the
`numba` package at module top and would fail collection, which is
cuSIMT's expected behavior (see NVIDIA/numba-cuda-mlir#136 — cuSIMT
intentionally does not depend on numba).
* CI: pin numpy<2.5 (mlir) and pytest<9.1 (cuda-core released tests)
Two nightly failure fixups after the first green iteration:
nightly-numba-cuda-mlir: numba-cuda-mlir 0.4.0 has an inverted guard
that registers an overload of np.row_stack on NumPy 2.x, and NumPy 2.5
removed that name entirely, so test collection fails with
"AttributeError: module 'numpy' has no attribute 'row_stack'". Cap
numpy to <2.5. See NVIDIA/numba-cuda-mlir#154.
nightly-cuda-core: released cuda-core v1.0.1's test suite uses a
parametrize argvalues pattern that pytest 9.1 rejects
("in parametrize the number of names (1)... must be equal to the
number of values (3)"). The main-side fix was #2212 but it has not
shipped in a cuda-core release yet. Cap pytest to <9.1 for the
released-cuda-core test run only.
* CI: deselect known pre-existing failures in nightly-cuda-core and nightly-numba-cuda-mlir
Applied only in the affected nightly-* pytest invocations; the released
source trees under test are unmodified.
nightly-numba-cuda-mlir (all 10 tests deselected are from cuSIMT):
* CudaArraySetting::{test_no_sync_default_stream, test_no_sync_supplied_stream, test_sync}
TestCudaArrayInterface::{test_consume_no_sync, test_consume_sync,
test_launch_no_sync, test_launch_sync,
test_launch_sync_two_streams, test_fortran_contiguous}
Serial-pytest contamination of numba_cuda_mlir.cuda.cudadrv from an
xfailed test in test_nrt_comprehensive.py. Upstream CI runs with
`pytest -n auto --dist loadscope`, which isolates the offending
side effect in a separate xdist worker; our nightly runs serially
and hits the pollution. See NVIDIA/numba-cuda-mlir#135.
* TestLinkerDumpAssembly::test_nvjitlink_jit_with_linkable_code_lto_dump_assembly_warn
Subprocess-invokes `cuobjdump`, which isn't on PATH in the base
ubuntu:24.04 container. Filed as an upstream skip-guard bug.
nightly-cuda-core (3 tests deselected are pre-existing v1.0.1 issues):
* test_enum_coverage.py::test_wrapper_covers_all_binding_members[NvlinkVersion]
Expected drift: main cuda-bindings adds NvlinkVersion.VERSION_6_0
which v1.0.1's wrapper mapping predates. This mode intentionally
pairs released core with main bindings, so this coverage-style
test will stay red here until a cuda-core release catches up.
* test_rlcompleter_patch.py::test_opt_out_env_var_disables_patch_even_when_interactive
Environment-dependent test: expects rlcompleter to crash without
the tab-completion patch, but on Windows MCDM the pre-patch
behavior is clean. Passes on Linux, fails on Windows MCDM.
* test_memory.py::test_non_managed_resources_report_not_managed[pinned]
Same underlying "Failed to allocate memory from pool" error that
v1.0.1 already xfails in the sibling test_pinned_memory_resource_initialization
(TODO(#9999)). cuda-python main has since fixed the parametrized
case to route through _allocate_pinned_buffer_or_xfail(), but that
fix hasn't shipped in a cuda-core release yet.
* CI: tighten deselects to per-platform failing sets
Previously applied the same list on both Linux and Windows workflows,
which over-deselected — some tests only fail on one platform because
the underlying issues (serial-pytest test-order in mlir, MCDM-only
behavior in cuda-core) are platform-specific.
Now:
nightly-numba-cuda-mlir
linux-64: TestCudaArrayInterface::{test_consume_no_sync,
test_consume_sync, test_launch_no_sync, test_launch_sync,
test_launch_sync_two_streams, test_fortran_contiguous}
+ TestLinkerDumpAssembly::test_nvjitlink_jit_with_linkable_code_lto_dump_assembly_warn.
win-64: CudaArraySetting::{test_no_sync_default_stream,
test_no_sync_supplied_stream, test_sync}
+ TestCudaArrayInterface::test_fortran_contiguous.
Test-order contamination in numba-cuda-mlir#135 surfaces different
tests depending on collection order (linux-64 vs win-64 exercise
different subsets), so the per-platform lists differ. cuobjdump-based
TestLinkerDumpAssembly only fires on Linux because the ubuntu:24.04
container's PATH lacks cuobjdump; Windows runners ship it with the
local CTK.
nightly-cuda-core
linux-64: test_enum_coverage.py::test_wrapper_covers_all_binding_members[NvlinkVersion].
win-64: NvlinkVersion (same as Linux)
+ test_rlcompleter_patch.py::test_opt_out_env_var_disables_patch_even_when_interactive
+ test_memory.py::test_non_managed_resources_report_not_managed[pinned].
rlcompleter and pinned mempool tests only fail on Windows MCDM.
NvlinkVersion fails on both (expected drift for the mode).
* CI: version-gate the nightly-mode deselects so they auto-clean
Each deselect is now wrapped in a bash conditional keyed on the
installed release version. When a newer numba-cuda-mlir or cuda-core
release ships with the referenced fix, the nightly picks it up
automatically, the guard evaluates false, and the deselect drops — so
the tests run against the new release. If they still fail we hear
about it loudly rather than silently masking a regression.
Current guards:
- numba-cuda-mlir #135 tests + cuobjdump TestLinkerDumpAssembly:
applied when installed numba-cuda-mlir version <= 0.4.0.
- cuda-core NvlinkVersion / rlcompleter opt-out / pinned mempool:
applied when installed cuda-core version <= 1.0.1.
Structure keeps one conditional block per (mode, platform) with a
comment above each deselect explaining the tracking issue.
* CI: broaden mlir deselect list to full #135 union across platforms
The previous per-platform-tight lists were incomplete: NVIDIA/numba-cuda-mlir#135's
import-time contamination poisons whichever tests reference
cuda.cudadrv.driver AFTER the polluting xfail runs, and collection
order varies between runs. Two consecutive Windows CI runs failed on
different subsets (3 slicing tests one run, 5 interface tests the
next).
Deselect the full union of #135-listed tests + test_fortran_contiguous
(observed to hit the same contamination) on both Linux and Windows.
Same version guard (<= 0.4.0) still applies, so the whole block drops
automatically when a newer numba-cuda-mlir release ships with the
root-cause fix.
Linux keeps the extra cuobjdump deselect (Linux-only environment
issue).
* Revert "cuda_pathfinder: pin nvshmem to <3.7 (was previously excluding only 3.7.0)"
This reverts commit 2a42aa7.
* Revert "Temporarily add push trigger to ci-nightly.yml for testing"
This reverts commit a0ccd19.1 parent 8d2d1d2 commit 45c8b24
5 files changed
Lines changed: 353 additions & 15 deletions
File tree
- .github/workflows
- ci
- tools
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
209 | 209 | | |
210 | 210 | | |
211 | 211 | | |
| 212 | + | |
| 213 | + | |
| 214 | + | |
| 215 | + | |
| 216 | + | |
| 217 | + | |
| 218 | + | |
| 219 | + | |
| 220 | + | |
| 221 | + | |
| 222 | + | |
| 223 | + | |
| 224 | + | |
| 225 | + | |
| 226 | + | |
| 227 | + | |
| 228 | + | |
| 229 | + | |
| 230 | + | |
| 231 | + | |
| 232 | + | |
| 233 | + | |
| 234 | + | |
| 235 | + | |
| 236 | + | |
| 237 | + | |
| 238 | + | |
| 239 | + | |
| 240 | + | |
| 241 | + | |
| 242 | + | |
| 243 | + | |
| 244 | + | |
| 245 | + | |
| 246 | + | |
| 247 | + | |
| 248 | + | |
| 249 | + | |
| 250 | + | |
| 251 | + | |
| 252 | + | |
| 253 | + | |
| 254 | + | |
| 255 | + | |
| 256 | + | |
| 257 | + | |
| 258 | + | |
| 259 | + | |
| 260 | + | |
| 261 | + | |
| 262 | + | |
| 263 | + | |
| 264 | + | |
| 265 | + | |
| 266 | + | |
| 267 | + | |
| 268 | + | |
| 269 | + | |
| 270 | + | |
| 271 | + | |
| 272 | + | |
| 273 | + | |
| 274 | + | |
| 275 | + | |
| 276 | + | |
| 277 | + | |
| 278 | + | |
| 279 | + | |
| 280 | + | |
| 281 | + | |
| 282 | + | |
| 283 | + | |
| 284 | + | |
| 285 | + | |
| 286 | + | |
| 287 | + | |
212 | 288 | | |
213 | 289 | | |
214 | 290 | | |
| |||
244 | 320 | | |
245 | 321 | | |
246 | 322 | | |
| 323 | + | |
| 324 | + | |
| 325 | + | |
| 326 | + | |
247 | 327 | | |
248 | 328 | | |
249 | 329 | | |
| |||
270 | 350 | | |
271 | 351 | | |
272 | 352 | | |
| 353 | + | |
| 354 | + | |
| 355 | + | |
| 356 | + | |
| 357 | + | |
| 358 | + | |
| 359 | + | |
| 360 | + | |
273 | 361 | | |
274 | 362 | | |
275 | 363 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
37 | 37 | | |
38 | 38 | | |
39 | 39 | | |
40 | | - | |
41 | | - | |
| 40 | + | |
| 41 | + | |
| 42 | + | |
42 | 43 | | |
43 | 44 | | |
44 | 45 | | |
| |||
409 | 410 | | |
410 | 411 | | |
411 | 412 | | |
| 413 | + | |
| 414 | + | |
| 415 | + | |
| 416 | + | |
| 417 | + | |
| 418 | + | |
| 419 | + | |
| 420 | + | |
| 421 | + | |
| 422 | + | |
| 423 | + | |
| 424 | + | |
| 425 | + | |
| 426 | + | |
412 | 427 | | |
413 | 428 | | |
414 | 429 | | |
| |||
420 | 435 | | |
421 | 436 | | |
422 | 437 | | |
| 438 | + | |
| 439 | + | |
| 440 | + | |
| 441 | + | |
| 442 | + | |
| 443 | + | |
| 444 | + | |
| 445 | + | |
| 446 | + | |
| 447 | + | |
| 448 | + | |
| 449 | + | |
| 450 | + | |
| 451 | + | |
| 452 | + | |
| 453 | + | |
| 454 | + | |
| 455 | + | |
| 456 | + | |
| 457 | + | |
| 458 | + | |
| 459 | + | |
| 460 | + | |
| 461 | + | |
| 462 | + | |
| 463 | + | |
| 464 | + | |
| 465 | + | |
| 466 | + | |
| 467 | + | |
| 468 | + | |
| 469 | + | |
| 470 | + | |
| 471 | + | |
| 472 | + | |
| 473 | + | |
| 474 | + | |
| 475 | + | |
| 476 | + | |
| 477 | + | |
| 478 | + | |
| 479 | + | |
| 480 | + | |
| 481 | + | |
| 482 | + | |
| 483 | + | |
| 484 | + | |
| 485 | + | |
| 486 | + | |
| 487 | + | |
| 488 | + | |
| 489 | + | |
| 490 | + | |
| 491 | + | |
| 492 | + | |
| 493 | + | |
| 494 | + | |
| 495 | + | |
| 496 | + | |
| 497 | + | |
| 498 | + | |
| 499 | + | |
| 500 | + | |
| 501 | + | |
| 502 | + | |
| 503 | + | |
| 504 | + | |
| 505 | + | |
| 506 | + | |
| 507 | + | |
| 508 | + | |
| 509 | + | |
| 510 | + | |
| 511 | + | |
| 512 | + | |
| 513 | + | |
| 514 | + | |
| 515 | + | |
| 516 | + | |
| 517 | + | |
| 518 | + | |
| 519 | + | |
| 520 | + | |
| 521 | + | |
| 522 | + | |
| 523 | + | |
| 524 | + | |
| 525 | + | |
| 526 | + | |
| 527 | + | |
| 528 | + | |
| 529 | + | |
| 530 | + | |
| 531 | + | |
| 532 | + | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
37 | 37 | | |
38 | 38 | | |
39 | 39 | | |
40 | | - | |
41 | | - | |
| 40 | + | |
| 41 | + | |
| 42 | + | |
42 | 43 | | |
43 | 44 | | |
44 | 45 | | |
| |||
388 | 389 | | |
389 | 390 | | |
390 | 391 | | |
| 392 | + | |
| 393 | + | |
| 394 | + | |
| 395 | + | |
| 396 | + | |
| 397 | + | |
| 398 | + | |
| 399 | + | |
| 400 | + | |
| 401 | + | |
| 402 | + | |
| 403 | + | |
| 404 | + | |
| 405 | + | |
| 406 | + | |
| 407 | + | |
391 | 408 | | |
392 | 409 | | |
393 | 410 | | |
| |||
401 | 418 | | |
402 | 419 | | |
403 | 420 | | |
| 421 | + | |
| 422 | + | |
| 423 | + | |
| 424 | + | |
| 425 | + | |
| 426 | + | |
| 427 | + | |
| 428 | + | |
| 429 | + | |
| 430 | + | |
| 431 | + | |
| 432 | + | |
| 433 | + | |
| 434 | + | |
| 435 | + | |
| 436 | + | |
| 437 | + | |
| 438 | + | |
| 439 | + | |
| 440 | + | |
| 441 | + | |
| 442 | + | |
| 443 | + | |
| 444 | + | |
| 445 | + | |
| 446 | + | |
| 447 | + | |
| 448 | + | |
| 449 | + | |
| 450 | + | |
| 451 | + | |
| 452 | + | |
| 453 | + | |
| 454 | + | |
| 455 | + | |
| 456 | + | |
| 457 | + | |
| 458 | + | |
| 459 | + | |
| 460 | + | |
| 461 | + | |
| 462 | + | |
| 463 | + | |
| 464 | + | |
| 465 | + | |
| 466 | + | |
| 467 | + | |
| 468 | + | |
| 469 | + | |
| 470 | + | |
| 471 | + | |
| 472 | + | |
| 473 | + | |
| 474 | + | |
| 475 | + | |
| 476 | + | |
| 477 | + | |
| 478 | + | |
| 479 | + | |
| 480 | + | |
| 481 | + | |
| 482 | + | |
| 483 | + | |
| 484 | + | |
| 485 | + | |
| 486 | + | |
| 487 | + | |
| 488 | + | |
| 489 | + | |
| 490 | + | |
| 491 | + | |
| 492 | + | |
| 493 | + | |
| 494 | + | |
| 495 | + | |
| 496 | + | |
| 497 | + | |
| 498 | + | |
| 499 | + | |
| 500 | + | |
| 501 | + | |
| 502 | + | |
0 commit comments