Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #1204 +/- ##
==========================================
+ Coverage 61.80% 63.12% +1.31%
==========================================
Files 61 65 +4
Lines 11880 12257 +377
==========================================
+ Hits 7343 7737 +394
+ Misses 4537 4520 -17 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
I'm not sure we want to do that, it's probably a better idea to defer to manual action in this case as it indicates that something went wrong (I'd expect that in most cases, manual action in question would be to rebase the PR, as that likely indicates that the change is too old) |
2a6b933 to
f649f61
Compare
|
@aduh95 reverted. |
f649f61 to
62688b3
Compare
Check Jenkins' context menu before scanning failures or posting a resume. When the action is missing, follow recorded resume ancestors for the same PR and approved commit, checking failures from every run along the way. Report unavailable actions and request failures with recovery guidance. Signed-off-by: Filip Skokan <panva.ip@gmail.com> Assisted-by: Codex
Check resume availability before suggesting a recovery action. Keep same-commit duplicates blocked when neither the latest run nor an eligible ancestor can resume, and guide users to inspect the run and recover manually. Suggest resume-ci only when a resume action is available. Signed-off-by: Filip Skokan <panva.ip@gmail.com> Assisted-by: Codex
Check Jenkins shutdown state and PR job readiness before processing CI requests. Report unavailable or unknown states with a nonzero exit code. Signed-off-by: Filip Skokan <panva.ip@gmail.com> Assisted-by: Codex
62688b3 to
7512639
Compare
Count running and queued PR CI jobs for workload threshold checks. Include PR builds waiting for downstream tests and count requests only once when they move from the queue to an executor between reads. Signed-off-by: Filip Skokan <panva.ip@gmail.com> Assisted-by: Codex
Print the matching diagnostic and console log URL when a changed file prevents resuming CI. Bound large excerpts and preserve early download cancellation. Assisted-by: Codex Signed-off-by: Filip Skokan <panva.ip@gmail.com>
A failed make target names its Makefile even when an unrelated test failed. Exclude these summaries from file attribution while retaining direct Makefile diagnostics and failures in changed tests. Assisted-by: Codex Signed-off-by: Filip Skokan <panva.ip@gmail.com>
7512639 to
2c640e7
Compare
This is what that comment in https://redirect.github.com/nodejs/node/pull/65879 would like now. Failed to resume CIFull Auto Start CI output |
|
--check-for-duplicatesblocking duplicate runs for the same approved commit. Suggestresume-ciorncu-ci resumewhen a resume action is available, and provide manual recovery guidance otherwise.ncu-ci availableto check whether Jenkins is quieting down and whethernode-test-pull-requestis enabled and buildable.ncu-ci workloadto print the number of running and queued PR jobs, including builds waiting for downstream tests, without double-counting requests moving between the queue and executors.makeexit summaries from file attribution while retaining direct Makefile diagnostics and failures in changed tests.The two additions are aimed at allowing node's auto-start-ci to skip piling on more jobs and exiting gracefully before touching labels if CI is unavailable.