Skip to content

Hide Prometheus integration when `remove_monitor_metrics` is true

What does this MR do and why?

Describe in detail what your merge request does and why.

This is part of hiding Monitor Metrics related views. The aim is deprecate the feature with the use of a feature flag and than proceed with the actual removal. This MR includes:

  • hide Settings > Integrations > Prometheus

Screenshots or screen recordings

Screenshots are required for UI changes, and strongly recommended for all other merge requests.

feature flag disabled feature flag enabled
Screenshot_2023-04-20_at_14.39.05 Screenshot_2023-04-20_at_14.40.15
Screenshot_2023-05-11_at_00.50.22 Screenshot_2023-05-11_at_00.48.08

How to set up and validate locally

Numbered steps to set up and validate the change are strongly suggested.

This part doesn't require any specific setup.

Check this view:

  • Settings > Integrations (project level)
  • Settings > Integrations (group level)
  • Settings > Integrations (admin level)

Prometheus should be visible on the integrations list when the flag is set to false, and is should not be on the list when flag is set to true

To enable the flag type this into terminal:

rails c
Feature.enable(:remove_monitor_metrics) 

MR acceptance checklist

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

Related to #399231 (closed)

Edited by Tristan Read

Merge request reports