Skip to content

ci: Copy nightly matrix into override to test GH UI loading#9209

Open
vyasr wants to merge 1 commit into
NVIDIA:mainfrom
vyasr:test/minimize_nightly
Open

ci: Copy nightly matrix into override to test GH UI loading#9209
vyasr wants to merge 1 commit into
NVIDIA:mainfrom
vyasr:test/minimize_nightly

Conversation

@vyasr
Copy link
Copy Markdown
Contributor

@vyasr vyasr commented Jun 1, 2026

Problem

The nightly CI workflow has become too large for the GitHub UI to load/render.

Approach

Copy the full nightly matrix (65 entries) into the workflows.override section so it runs as a PR workflow. This lets us observe whether the GH UI can handle the nightly job count, and iteratively reduce it to find a manageable subset.

Important Notes

  • This PR is intentionally non-mergeable — the override section blocks merging by design.
  • The actual nightly workflow is unaffected (it doesn't use --allow-override).
  • Once a working subset is identified (or the diagnosis is complete), the override will be reset to empty.
  • Commit uses [skip-vdc][skip-docs][skip-tpt] to reduce unrelated CI noise during this experiment.

@vyasr vyasr requested a review from a team as a code owner June 1, 2026 21:48
@vyasr vyasr requested a review from jrhemstad June 1, 2026 21:48
@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: dce940fd-4d98-4c1d-98a6-186033ca0df7

📥 Commits

Reviewing files that changed from the base of the PR and between b01ac23 and 708c815.

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

📝 Walkthrough

Summary by CodeRabbit

  • Chores
    • Expanded continuous integration to broaden build and test coverage across multiple CTK versions, GPU targets and architectures (including arm64), add GPU-specific checks, packaging and install tests, Python test entries, compiler/tooling builds and linting, and additional architecture-specific build groups to improve overall platform reliability.

suggestion:

Walkthrough

The PR expands ci/matrix.yaml's workflows.override matrix from a single entry to comprehensive enumeration of CI job configurations. Coverage now spans CTK versions 12.0, 12.X, 13.0, and 13.X across default projects and cudax builds, multi-target GPU testing with limited RTX PRO 6000 access, specialized configurations for arm64/thrust/NVRTC/C bindings, Python testing, packaging validation, and compiler-specific builds (NVHPC, clang-cuda, clang-tidy) with architecture-specific sm grouping.

Changes

CI Matrix Expansion for CTK and Compiler Coverage

Layer / File(s) Summary
Build matrix for default projects and cudax
ci/matrix.yaml
Expanded build matrix entries for CTK 12.0, 12.X, 13.0, and 13.X targeting both default projects and cudax, each with compiler/standard groupings.
GPU testing matrices with hardware coverage
ci/matrix.yaml
CTK 12.X and 13.X testing matrices covering multi-project GPU tests and limited RTX PRO 6000 target coverage via test_nolid and test_lid0 variants.
Specialized test and build configurations
ci/matrix.yaml
Arm64 build coverage, thrust GPU dispatch mode checks, NVRTC/verify_codegen jobs with CTK support constraints, and cccl_c_parallel/cccl_c_stf pinned matrices with limited RTX PRO 6000 coverage.
Python test and packaging matrix additions
ci/matrix.yaml
Python test override entries with Linux-consistency CTK/py_version/GPU/compiler configurations, plus CCCL packaging test/install entries and NVBench helper test matrices across CTK and host compiler combinations.
Compiler-specific and architecture-focused builds
ci/matrix.yaml
NVHPC, clang-cuda (with sm list coverage), clang-tidy, and architecture/family-specific sm build group entries for selected projects.

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

@github-actions

This comment has been minimized.

…][skip-docs]

The nightly CI workflow has become too large for the GitHub UI to
load. To diagnose and find a manageable subset, copy the full nightly
matrix (65 entries) into the override section so it runs as a PR
workflow for observation.

This override blocks merging by design. Once a working subset is
identified, the override will be reset to empty.
@vyasr vyasr force-pushed the test/minimize_nightly branch from b01ac23 to 708c815 Compare June 2, 2026 05:00
@github-actions

This comment has been minimized.

@jrhemstad
Copy link
Copy Markdown
Collaborator

/ok to test

@jrhemstad
Copy link
Copy Markdown
Collaborator

The StepSecurity reports should be disabled now. Kicked off another run to see if that fixes the issue.

@jrhemstad
Copy link
Copy Markdown
Collaborator

/ok to test 708c815

@jrhemstad jrhemstad closed this Jun 4, 2026
@github-project-automation github-project-automation Bot moved this from In Review to Done in CCCL Jun 4, 2026
@jrhemstad jrhemstad reopened this Jun 4, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jun 4, 2026

😬 CI Workflow Results

🟥 Finished in 1h 36m: Pass: 90%/1054 | Total: 8d 20h | Max: 58m 58s | Hits: 99%/1196940

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

Archived in project

Development

Successfully merging this pull request may close these issues.

2 participants