Repository navigation
Update CI failure issue template for objective reports - #8720
Conversation
|
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. |
|
Navigate logical layers of code changes, visualize relationships, and explore their blast radius. No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository: NVIDIA/cuml/.coderabbit.yaml Review profile: CHILL Plan: Enterprise Run ID: 📒 Files selected for processing (1)
Included review availability: This review used your included allowance. Your plan provides up to 12 included reviews per hour; 11 remain after this review. 📝 SummarySummary by CodeRabbit
WalkthroughThe CI failure issue template now collects objective failure information, job links, failure details, environment values, and investigation follow-up. ChangesCI failure issue reporting
Priority: ⬇️ Low Estimated code review effort: 1 (Trivial) | ~5 minutes Change: Other Merge Risk: ⚪ Minimal · up to No actionable merge-blocking risk is evident from the supplied context. The template directs unresolved bugs to separate tracking. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
betatim
left a comment
There was a problem hiding this comment.
Removing the need for a root cause makes it easier to just file an issue when we see a CI failure. I think that is a good thing.
By the time I have understood the cause of a failure it is as much work to submit a PR that fixes it as it is to file an issue. So removing the need for the root cause makes it easier to file the issue first. Which I think we want for tracking purposes right?
Yes, exactly. The RCA was always optional, but the revised template is even more clear that this should just be an objective report, no RCA needed and even slightly discouraged if you will. |
|
/merge |
Updates the CI failure issue template to focus on the observed failure and exact failing job. Removes the root-cause section, clarifies how to omit inapplicable fields, and asks maintainers to track unresolved bugs separately when a CI failure is mitigated.