Skip to content

Test GB300 runners - #2483

Closed
msarahan wants to merge 4 commits into
mainfrom
test-gb300
Closed

Test GB300 runners#2483
msarahan wants to merge 4 commits into
mainfrom
test-gb300

Conversation

@msarahan

Copy link
Copy Markdown
Contributor

Use the shared-workflows test-gb300 branch to exercise Grace+Blackwell, mirroring the existing GH200 test PR. Requires rapidsai/shared-workflows#591 and nv-gha-runners/enterprise-runner-configuration#401.\n\nValidation: pre-commit run --files .

@copy-pr-bot

copy-pr-bot Bot commented Jul 13, 2026

Copy link
Copy Markdown

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.

@coderabbitai

coderabbitai Bot commented Jul 13, 2026

Copy link
Copy Markdown

Review Change Stack

Note

Reviews paused

It looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the reviews.auto_review.auto_pause_after_reviewed_commits setting.

Use the following commands to manage reviews:

  • @coderabbitai resume to resume automatic reviews.
  • @coderabbitai review to trigger a single review.

Use the checkboxes below for quick actions:

  • ▶️ Resume reviews
  • 🔍 Trigger review
📝 Walkthrough

Summary by CodeRabbit

  • Chores
    • Updated CI workflows to pin reusable workflow references to test-gb300 instead of main, including build, test, docs, wheels, and devcontainer pipelines.
    • Bumped rapidsai/ci-conda to 26.10-latest for docs and C++ debug test runs.
    • Adjusted file-change detection to also exclude !.yamllint.yaml.
    • Added branch-gated API documentation publishing and a CMake test job to the PR pipeline.
    • Increased C++ test timeout from 15m to 30m.

Walkthrough

Reusable GitHub Actions workflows are retargeted from @main to @test-gb300 across build, pull-request, test, and breaking-change alert workflows. CMake testing and API documentation publishing jobs are added, selected container images are updated, and the C++ test timeout is extended.

Changes

Workflow updates

Layer / File(s) Summary
Build and publishing pipeline
.github/workflows/build.yaml
Build, documentation, package upload, wheel, and devcontainer jobs use @test-gb300; branch builds add API documentation publishing.
Pull-request validation and documentation
.github/workflows/pr.yaml
Pull-request workflows use @test-gb300 for shared jobs and add CMake testing plus dry-run API documentation publishing.
Test and breaking-change workflows
.github/workflows/test.yaml, .github/workflows/trigger-breaking-change-alert.yaml
Test jobs and breaking-change notifications use @test-gb300, with the debug test container updated.
C++ test timeout
ci/test_cpp.sh
The gtest execution timeout increases from 15 minutes to 30 minutes while preserving exit-code handling.

Estimated code review effort: 2 (Simple) | ~10 minutes

Possibly related PRs

Suggested labels: ci, improvement

Suggested reviewers: gforsyth, jameslamb

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly matches the main change: switching workflows to test GB300 runners.
Description check ✅ Passed The description is directly related to the workflow updates and GB300 runner testing.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches 💡 1
🛠️ Fix failing CI checks 💡
  • Fix failing CI checks
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch test-gb300

Comment @coderabbitai help to get the list of available commands.

@copy-pr-bot

copy-pr-bot Bot commented Jul 13, 2026

Copy link
Copy Markdown

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.

@KyleFromNVIDIA

Copy link
Copy Markdown
Member

/ok to test f2d06f3

@KyleFromNVIDIA

Copy link
Copy Markdown
Member

/ok to test e15b27b

@KyleFromNVIDIA

Copy link
Copy Markdown
Member

/ok to test 080466e

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants