Skip to content

ci: pinned-memory diagnostics with isolated wheel imports - #3458

Draft
jpascucci-nv wants to merge 4 commits into
NVIDIA:mainfrom
jpascucci-nv:issue3426-2
Draft

jpascucci-nv wants to merge 4 commits into
NVIDIA:mainfrom
jpascucci-nv:issue3426-2

Conversation

@jpascucci-nv

Copy link
Copy Markdown
Contributor

Description

Run dedicated Windows L4 diagnostics for pinned-memory failures using the wheels selected by CI. Launch pytest in isolated Python processes so the source checkout cannot shadow the installed cuda.core wheel and cause the missing cuda.core._version import error.

Verify wheel imports, reject all-skipped diagnostic runs, and preserve per-case logs, JUnit results, and first-OOM reports.

Validation: 46 targeted tests passed; Ruff and actionlint passed. GPU validation pending CDCI.

Checklist

  • I have read the Contributing Guidelines.
  • New or existing tests cover these changes.
  • The documentation is up to date with these changes.

Run dedicated Windows L4 diagnostics for pinned-memory failures using
the wheels selected by CI. Launch pytest in isolated Python processes
so the source checkout cannot shadow the installed cuda.core wheel
and cause the missing cuda.core._version import error.

Verify wheel imports, reject all-skipped diagnostic runs, and preserve
per-case logs, JUnit results, and first-OOM reports.

Validation: 46 targeted tests passed; Ruff and actionlint passed.
GPU validation pending CDCI.
@jpascucci-nv jpascucci-nv added this to the cuda.bindings next milestone Nov 13, 2026
@jpascucci-nv jpascucci-nv self-assigned this Nov 13, 2026
@jpascucci-nv jpascucci-nv added bug Something isn't working P1 Medium priority - Should do cuda.core Everything related to the cuda.core module labels Nov 13, 2026
@copy-pr-bot

copy-pr-bot Bot commented Nov 13, 2026

Copy link
Copy Markdown
Contributor

Auto-sync is disabled for draft pull requests in this repository. Workflows must be run manually.

Contributors can view more details about this message here.

@github-actions github-actions Bot added the CI/CD CI/CD infrastructure label Nov 13, 2026
@jpascucci-nv

Copy link
Copy Markdown
Contributor Author

/ok to test 9ee75ba

Compare legacy, per-thread, and nonblocking streams with and without
access revocation. Alternate watermark resets with minimal logging,
preserve the last API boundary across crashes, and time out hung probes.

Capture fixture OOM diagnostics before popping the CUDA context,
preserving the original failure and the first diagnostic snapshot.

Validation: 29 CPU checks, crash/hang checks, and Ruff passed.
GPU validation pending CDCI.
@jpascucci-nv

Copy link
Copy Markdown
Contributor Author

/ok to test 2a7d7b0

Build a standalone C++ driver-API probe to isolate failures from
Python bindings. Compare stream and access-revocation combinations,
plus controls omitting trim, omitting watermark reset, or reusing a pool.

Preserve first-failure diagnostics and progress through crashes and
timeouts, and upload the executable, source, build log, and results.

Validate with 22 focused tests and compilation against CUDA headers.
@jpascucci-nv

Copy link
Copy Markdown
Contributor Author

/ok to test 10b05ff

@jpascucci-nv

Copy link
Copy Markdown
Contributor Author

/ok to test e77c968

@jpascucci-nv

Copy link
Copy Markdown
Contributor Author

This branch has not been deployed

No deployments
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 P1 Medium priority - Should do

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant