check-nightly-ci: update to new version#5433
Conversation
jameslamb
added
improvement
|
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. |
|
/ok to test |
|
/ok to test |
| repo: ${{ github.repository }} | ||
| target-branch: ${{ fromJSON(steps.get-pr-info.outputs.pr-info).base.ref }} | ||
| # TODO: remove this when nightly tests are passing again (ref: https://github.com/rapidsai/cugraph/issues/5421) | ||
| max-days-without-success: 21 |
There was a problem hiding this comment.
Oh cool this set of PRs found a bug 😁
It looks like, due to the bug being fixed in , check-nightly-ci has been passing on PRs targeting main here for the last few weeks when it really shouldn't have been!!
On this PR:
The oldest run of workflow 'test.yaml' on branch 'main' was 14 days ago (2026-02-03 07:13:23+00:00).
There were 0 successful runs of workflow 'test.yaml' on branch 'main' in the last 7 days.
On other recent PRs:
The most recent successful run of the test.yaml workflow on release/26.02 was 2026-02-11 11:13:42+00:00, which is within the last 7 days. View logs:
test.yaml hasn't succeeded on main since February 5: https://github.com/rapidsai/cugraph/actions/workflows/test.yaml
But check-nightly-ci was passing because it last passed on release/26.02
The failures look like they're on the RTX Pro 6000 jobs: https://github.com/rapidsai/cugraph/actions/runs/21984468554/job/63515107657
I believe #5422 intentionally only skipped them in PR CI and left them failing for nightlies, but not sure (I didn't think to ask there when I reviewed). The work of fixing those is tracked here: #5421
For now, proposing bumping this up to 21 days. That'd give until February 26 (9 days from now) to fix this.
jameslamb
changed the title
|
admin-merging to get things working again, now that rapidsai/shared-actions#96 has been merged |
Follow-up to #5433 Removes some testing configuration left behind in that PR. Authors: - James Lamb (https://github.com/jameslamb) - Bradley Dice (https://github.com/bdice) Approvers: - Kyle Edwards (https://github.com/KyleFromNVIDIA) URL: #5436
2 participants
Contributes to rapidsai/shared-actions#94
Uses the updated
check_nightly_successcheck from rapidsai/shared-actions#96 . Now that CI check will only ever consider the branch a PR targets, which should prevent issues related to release timing in the future.Notes for Reviewers
I've intentionally cancelled all other CI here after
check-nightly-ciandchecksran, as this only affectscheck-nightly-ci.Merging plan:
@mainhere