stolostron/capi-tests: use conditional Slack report template for capz-e2e#80824
stolostron/capi-tests: use conditional Slack report template for capz-e2e#80824RadekCap wants to merge 1 commit into
Conversation
…-e2e Add success state reporting and use Go template conditionals to show different messages per job state: green for success, failed emoji for failure/error. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository YAML (base), Central YAML (inherited) Review profile: CHILL Plan: Enterprise Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (1)
WalkthroughThe ChangesProw capz-e2e Slack Reporting
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes 🚥 Pre-merge checks | ✅ 15✅ Passed checks (15 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ 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 |
|
@RadekCap: all tests passed! Full PR test history. Your PR dashboard. DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here. |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: marek-veber, mzazrivec, RadekCap The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Summary
successtojob_states_to_reportfor the capz-e2e periodic job{{if eq .Status.State "success"}}) to show:slack-green:for success and:failed:for failure/error🤖 Generated with Claude Code
Summary by CodeRabbit
This PR updates the Slack notification configuration for the
capz-e2eperiodic CI job in the stolostron/capi-tests repository to provide better visibility into job outcomes.Changes made:
job_states_to_reportto includesuccessstate alongside existing failure/error states, enabling notifications for both successful and failed job completions:slack-green:emoji when the job succeeds:failed:emoji when the job fails or errorsPractical impact: Team members monitoring the capz-e2e job in Slack will now receive status-specific visual indicators, making it immediately clear whether the job succeeded or failed without needing to check the detailed logs. This improves observability and makes it easier to spot job failures in busy Slack channels.