docs(ci): publish CI alert runbook and validation evidence#281
Conversation
|
Warning Review limit reached
Next review available in: 58 minutes Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available. How can I continue?After more reviews become available, a review can be triggered using the To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews. How do review limits work?CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability. For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window. Please refer docs for additional details. Review details⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (4)
📝 WalkthroughWalkthroughChangesCI alert documentation
Estimated code review effort: 1 (Trivial) | ~5 minutes Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
Nit: The runbook (line 53) and policy doc (line 44) both say "within 5 minutes of workflow failure," but for matrix workflows the alert waits for all jobs to finish before sending. The QG4 validation sample shows 6m37s from first failure to notification, but only 6s from workflow conclusion. Consider saying "within 5 minutes of workflow conclusion" so responders know what to expect for matrix workflows. Assisted by Claude Opus 4.6 (1M context) |
Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Clarify that payload quality checks use local render_payload.sh preview output and distinguish failing-step times from Jobs API job completedAt values in the evidence summary. Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
156a6ad to
f9eefde
Compare
Description
This PR completes
RHAIENG-5938by documenting and validating the existing shared-branch CI Slack alert flow without changing workflow behavior.Changes in this PR:
docs/ci-alert-runbook.mdas the evergreen responder runbook for CI alertsdocs/ci-alert-validation-report.mdwith live evidence from recentmainfailures and payload-quality verificationdocs/ci-alert-policy.mdanddocs/ci-health-dashboard.mdto link to the new runbook/evidence docsMotivation:
Jira Ticket
RHAIENG-5938
Testing
make testpasses (run from the affected agent directory)Additional verification:
uv run --no-sync python -m pytest .github/scripts/tests/test_notify_slack.py -v(12 passed)mainGitHub Actions evidence for:Inner Loop Gatingrun29016905658QG4: Agent Deployment Integration Testsrun29890165253.github/actions/notify-slack/render_payload.shto confirm:Checklist
.envor secret files are included in this PRReview Guidance
Please review these files in order:
docs/ci-alert-runbook.mddocs/ci-alert-validation-report.mdThe changes in
docs/ci-alert-policy.mdanddocs/ci-health-dashboard.mdare just cross-links to the new docs.Important scope note:
Related PRs