Skip to content

Bring back attributed notification to master-broken channel

Jennifer Li requested to merge jennli-restore-master-broken-notification into master

What does this MR do and why?

We need to undo Proposal: reduce notification urgency, or skip ... (gitlab-org/quality/engineering-productivity/team#235 - closed), because attributed groups might not have any triage DRI online in the event of urgent incidents, in which case we must ensure the alert goes to a publicly monitored channel. We have had 2 occurrences this week where action-required incidents went unnoticed for hours.

This MR restores all master broken incident alerts to #master-broken channel.

Announced this change in Slack thread.

Expected impact & dry-runs

These are strongly recommended to assist reviewers and reduce the time to merge your change.

See https://gitlab.com/gitlab-org/quality/triage-ops/-/tree/master/doc/scheduled#testing-policies-with-a-dry-run on how to perform dry-runs for new policies.

See https://gitlab.com/gitlab-org/quality/triage-ops/-/blob/master/doc/reactive/best_practices.md#use-the-sandbox-to-test-new-processors on how to make sure a new processor can be tested.

Action items

Merge request reports