Skip to content

Remove disabled alerts section from settings page

Rajendra Kadam requested to merge 338834-remove-disabled-alerts-prometheus into master

What does this MR do and why?

Remove the deprecation banner on Settings > Integrations > Prometheus

Remove the alerts section from Settings > Integrations > Prometheus.

Related to #338834 (closed)

Screenshots or screen recordings

Integration Settings > Prometheus Settings > Monitor Settings > Alerts
Screenshot_from_2022-05-04_13-07-36 Screenshot_from_2022-05-04_13-25-14

How to set up and validate locally

  1. Go to project's Integration Settings > Prometheus
  2. Check that the banner on top is not present anymore
  3. Check that "Alerts" section is not available anymore

However, alert integrations is left intact:

  1. Go to project's Settings > Monitor Settings > Alerts
  2. Check that managing alert integrations still works

We have left the documentation intact and will change it in a follow-up MR by following deprecations guidelines for documentation

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Edited by Rajendra Kadam

Merge request reports