Skip to content

CI: retry arm64 GH200 nightly-standard row + add GB300#2296

Open
leofang wants to merge 9 commits into
NVIDIA:mainfrom
leofang:leofang/retry-gh200-nightly
Open

CI: retry arm64 GH200 nightly-standard row + add GB300#2296
leofang wants to merge 9 commits into
NVIDIA:mainfrom
leofang:leofang/retry-gh200-nightly

Conversation

@leofang

@leofang leofang commented Jul 2, 2026

Copy link
Copy Markdown
Member

Re-enable the gh200 nightly-standard row in ci/test-matrix.yml (previously disabled in 4c70cfa because the runner hung on cudaMallocAsync). Runner team indicated the pool-side issue has been fixed, so let's give it another CI-visible run.

Also added access to gb300, which exposes a minor issue in the way we generate object/library files for testing.

Also fixed a few nightly CI issues due to the recent numba-cuda-mlir v0.4.1 release

leofang added 2 commits July 2, 2026 15:01
Reverts the disable in 4c70cfa now that the runner team has fixed the
pool-side hang on stream-ordered memory allocator calls.
@copy-pr-bot

copy-pr-bot Bot commented Jul 2, 2026

Copy link
Copy Markdown
Contributor

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@leofang

leofang commented Jul 2, 2026

Copy link
Copy Markdown
Member Author

/ok to test 8d51cf7

@github-actions github-actions Bot added the CI/CD CI/CD infrastructure label Jul 2, 2026
@github-actions

github-actions Bot commented Jul 2, 2026

Copy link
Copy Markdown

The cufile BAR-size query returns CUDA_ERROR_NOT_SUPPORTED on
Grace+Hopper (unified memory, no discrete PCIe BAR). Tracked in NVIDIA#2299;
remove this deselect once the test skipif is fixed upstream.

Uses PYTEST_ADDOPTS so no changes to run-tests are needed.
@leofang

leofang commented Jul 2, 2026

Copy link
Copy Markdown
Member Author

/ok to test f148214

Comment thread ci/test-matrix.yml Outdated
@leofang leofang changed the title CI: retry arm64 gh200 nightly-standard row CI: retry arm64 GH200 nightly-standard row + add GB300 Jul 13, 2026
@leofang

leofang commented Jul 13, 2026

Copy link
Copy Markdown
Member Author

/ok to test 5bbfa09

@leofang leofang added enhancement Any code-related improvements P1 Medium priority - Should do labels Jul 13, 2026
@leofang leofang added this to the cuda.bindings 13.4.0 & 12.9.8 milestone Jul 13, 2026
…3 / GB300)

-arch=all-major only emits X.0 cubins (sm_100, sm_90, ...), so nvJitLink returns ERROR_INVALID_INPUT when the RDC saxpy fixture is linked on GB300 (sm_103): Blackwell no longer serves an sm_10x cubin from the sm_100 image. -arch=all additionally emits sm_103 (and other minor archs) so the linker finds a usable input. The fixture is built in-job by build_test_binaries.py, so there is no committed artifact to regenerate.
@github-actions github-actions Bot added the cuda.core Everything related to the cuda.core module label Jul 13, 2026
leofang added 2 commits July 13, 2026 21:45
…deselect gate to 0.4.1

test_fortran_contiguous calls cp.random, which dlopens libcurand at runtime; the mlir nightly install didn't provide it, unlike the numba-cuda branch (which installs cuda-toolkit[curand,cublas] + cupy). Mirror that here.

Bump the version-gated mlir deselect from <=0.4.0 to <=0.4.1: 0.4.1 still fails the cuobjdump-invoking nvjitlink test (wheel envs lack cuobjdump) and the NVIDIA#135 pytest-contamination tests, so keep them deselected. cu12 rows remain red due to the separate NVIDIA#2320 LTO-IR regression (not in the deselect list).
@leofang

leofang commented Jul 13, 2026

Copy link
Copy Markdown
Member Author

/ok to test 7f44b44

"-dc",
*nvcc_extra_flags,
"-arch=all-major",
"-arch=all",

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should fix a bug on GB300 (cc 10.3). Unfortunately, we can't verify this in the PR because of the binary fetch logic.

@leofang leofang self-assigned this Jul 13, 2026
@leofang leofang added bug Something isn't working P0 High priority - Must do! and removed P1 Medium priority - Should do labels Jul 13, 2026
@leofang

leofang commented Jul 13, 2026

Copy link
Copy Markdown
Member Author

/ok to test 4b61b3c

Comment thread ci/test-matrix.yml
# allocator (cudaMallocAsync); runner pool needs fixing first.
# - { ARCH: 'arm64', PY_VER: '3.14', CUDA_VER: '13.3.0', LOCAL_CTK: '1', GPU: 'gh200', GPU_COUNT: '1', DRIVER: 'latest', ENV: { MODE: 'nightly-standard' } }
- { ARCH: 'arm64', PY_VER: '3.13', CUDA_VER: '13.3.0', LOCAL_CTK: '1', GPU: 'gh200', GPU_COUNT: '1', DRIVER: 'latest', ENV: { MODE: 'nightly-standard' } }
- { ARCH: 'arm64', PY_VER: '3.14', CUDA_VER: '13.3.0', LOCAL_CTK: '1', GPU: 'gb300', GPU_COUNT: '1', DRIVER: 'latest', ENV: { MODE: 'nightly-standard' } }

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These two lines are real, non-cosmetic changes (gh200/gb300).

@leofang leofang marked this pull request as ready for review July 13, 2026 23:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working CI/CD CI/CD infrastructure cuda.core Everything related to the cuda.core module enhancement Any code-related improvements P0 High priority - Must do!

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant