Skip to content

Also test Thrust on SM120 in nightly/weekly CI#9201

Open
bernhardmgruber wants to merge 1 commit into
NVIDIA:mainfrom
bernhardmgruber:test_thrust_sm120
Open

Also test Thrust on SM120 in nightly/weekly CI#9201
bernhardmgruber wants to merge 1 commit into
NVIDIA:mainfrom
bernhardmgruber:test_thrust_sm120

Conversation

@bernhardmgruber
Copy link
Copy Markdown
Contributor

@bernhardmgruber bernhardmgruber commented Jun 1, 2026

I would love to also have it in the per pull request CI, but I guess @alliepiper tried to use the rtxpro6000 runners sparingly.

Fixes: #8877

@bernhardmgruber bernhardmgruber requested a review from a team as a code owner June 1, 2026 11:12
@bernhardmgruber bernhardmgruber requested a review from jrhemstad June 1, 2026 11:12
@github-project-automation github-project-automation Bot moved this to Todo in CCCL Jun 1, 2026
@cccl-authenticator-app cccl-authenticator-app Bot moved this from Todo to In Review in CCCL Jun 1, 2026
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Jun 1, 2026

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Enterprise

Run ID: b5c5d4ec-4b65-4376-9765-a60b7937ef5e

📥 Commits

Reviewing files that changed from the base of the PR and between 0db32d9 and b942bfa.

📒 Files selected for processing (1)
  • ci/matrix.yaml
🚧 Files skipped from review as they are similar to previous changes (1)
  • ci/matrix.yaml

Note: CodeRabbit is enabled on this repository as a convenience for maintainers
and contributors. Use your best judgment when considering its review comments and
suggestions — a suggested change may be inadequate, unnecessary, or safe to ignore.
Contributors are not expected to address every comment. Human reviews are what
ultimately matter for merging.

Summary by CodeRabbit

  • Chores
    • Expand CI test coverage for RTX PRO 6000 hardware: in ci/matrix.yaml the RTX PRO 6000 entries for both nightly and weekly workflows now run project ['cub', 'thrust'] instead of just 'cub', adding Thrust test runs on SM120 to nightly/weekly CI. This is a CI-only change (+2/-2) intended to detect regressions like the NVBug 6156669 regression fixed in PR #8876 and addresses the request in Issue #8877.

suggestion:

Walkthrough

CI matrix configuration expanded to run Thrust tests on RTX PRO 6000 (SM120) alongside CUB tests in nightly and weekly workflows; two matrix entries changed project from cub to ['cub','thrust'].

Changes

RTX PRO 6000 Thrust Coverage

Layer / File(s) Summary
CI matrix Thrust expansion
ci/matrix.yaml
Updated nightly and weekly workflow RTX PRO 6000 coverage matrix to test both CUB and Thrust projects instead of CUB only.

Assessment against linked issues

Objective Addressed Explanation
Add Thrust CI testing on SM120 [#8877]

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

@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@bernhardmgruber
Copy link
Copy Markdown
Contributor Author

/ok to test 042d88d

@bernhardmgruber
Copy link
Copy Markdown
Contributor Author

/ok to test 0db32d9

@github-actions

This comment has been minimized.

@bernhardmgruber
Copy link
Copy Markdown
Contributor Author

/ok to test c5f6969

@github-actions

This comment has been minimized.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jun 3, 2026

😬 CI Workflow Results

🟥 Finished in 2h 04m: Pass: 98%/500 | Total: 5d 21h | Max: 1h 00m | Hits: 85%/748083

See results here.

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

Labels

None yet

Projects

Status: In Review

Development

Successfully merging this pull request may close these issues.

Please test Thrust on SM120

1 participant