Skip to content

feat: combine gprd main and cny pages

Steve Xuereb requested to merge feat/combine-cny-main-alerts into master

What

  • Combine the pagerduty pages for stage when it's main and cny
  • Delete the now unused RouteCase method.
Before After
Screenshot_2022-07-08_at_16.28.32
Screenshot_2022-07-08_at_16.16.08
Screenshot_2022-07-08_at_16.27.32
Screenshot_2022-07-08_at_16.15.10

Why

In https://gitlab.com/gitlab-com/gl-infra/reliability/-/issues/15765 we've started grouping alerts by the service this resulsted into group the alerts with that are firing for cny and main stages which is what we want. However we end up paging the on-call twice because we have 2 recivers as show in https://gitlab.com/gitlab-com/gl-infra/reliability/-/issues/15964#problem.

We already show the stage if it's main or cny in the alert labels and in the alert description so it's already pretty clear wich stage is effected.

Reference: https://gitlab.com/gitlab-com/gl-infra/reliability/-/issues/15964

Merge request reports