Skip to content

bump nightly test to 26.06#1091

Merged
rapids-bot[bot] merged 1 commit intoNVIDIA:mainfrom
rgsl888prabhu:update_nightly_workflow
Apr 14, 2026
Merged

bump nightly test to 26.06#1091
rapids-bot[bot] merged 1 commit intoNVIDIA:mainfrom
rgsl888prabhu:update_nightly_workflow

Conversation

@rgsl888prabhu
Copy link
Copy Markdown
Collaborator

Description

Bumpy nightly tests for 26.06 and disable 26.04

Checklist

  • I am familiar with the Contributing Guidelines.
  • Testing
    • New or existing tests cover these changes
    • Added tests
    • Created an issue to follow-up
    • NA
  • Documentation
    • The documentation is up to date with these changes
    • Added new documentation
    • NA

@rgsl888prabhu rgsl888prabhu requested a review from a team as a code owner April 10, 2026 15:41
@rgsl888prabhu rgsl888prabhu requested a review from vyasr April 10, 2026 15:41
@rgsl888prabhu rgsl888prabhu self-assigned this Apr 10, 2026
@rgsl888prabhu rgsl888prabhu added non-breaking Introduces a non-breaking change improvement Improves an existing functionality labels Apr 10, 2026
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 10, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 7fd2ab13-feed-43b9-a5dc-18b019745704

📥 Commits

Reviewing files that changed from the base of the PR and between 24fdb60 and a0a0b40.

📒 Files selected for processing (1)
  • .github/workflows/nightly.yaml

📝 Walkthrough

Walkthrough

Updated GitHub Actions workflow job matrix to replace the cuopt_branch matrix entry from release/26.04 to release/26.06, changing which release branch is referenced when the nightly workflow dispatches build.yaml via the CUOPT_BRANCH parameter.

Changes

Cohort / File(s) Summary
GitHub Actions Workflow Configuration
.github/workflows/nightly.yaml
Updated cuopt_branch matrix entry from release/26.04 to release/26.06

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and specifically describes the main change: updating nightly tests to use version 26.06.
Description check ✅ Passed The description is related to the changeset, indicating the intent to update nightly CI to use 26.06 and disable 26.04, which aligns with the workflow file changes.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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

@rgsl888prabhu
Copy link
Copy Markdown
Collaborator Author

/merge

@rapids-bot rapids-bot Bot merged commit 90e37ab into NVIDIA:main Apr 14, 2026
208 of 211 checks passed
rapids-bot Bot pushed a commit that referenced this pull request May 7, 2026
## Summary

Make the two matrix entries in `.github/workflows/nightly.yaml` independent so a missing release-branch ref no longer cancels the working `main`-branch nightly.

**Why this matters now:** the matrix bumped to `release/26.06` in #1091 on 2026-04-14, but `release/26.06` has not been cut from `main` yet. Every nightly since has failed — the release entry 404s at the `gh api .../branches/release/26.06` lookup, the matrix's default `fail-fast: true` cancels the sibling `main` entry, and the wrapper job reports `failure` even though the `main` dispatch would have succeeded on its own.

**Fix:** add `fail-fast: false` so each cuopt_branch dispatch runs independently.

```yaml
strategy:
  fail-fast: false       # ← added
  matrix:
    cuopt_branch:
      - "main"
      - "release/26.06"
```

The release entry will still go red until `release/26.06` is actually cut (or the line is reverted to a release branch that exists), but it no longer drags `main` with it.

## Test plan
- [ ] Watch the next scheduled nightly — `main` matrix entry should dispatch and complete independent of the `release/26.06` entry's outcome.
- [ ] Manual dispatch via `Run workflow` from Actions UI to verify both entries run to completion (or independent failure) before waiting for the cron.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Authors:
  - Ramakrishnap (https://github.com/rgsl888prabhu)

Approvers:
  - James Lamb (https://github.com/jameslamb)

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

Labels

improvement Improves an existing functionality non-breaking Introduces a non-breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants