Skip to content

Remove/Update alerts controller, specs and prometheus proxy class

Rajendra Kadam requested to merge 397138-metric-api-helper-removals-2 into master

What does this MR do and why?

Aids in helping delete/remove files related to #397138 (closed)

In the removals you can see that majority of code is around the feature flag that is removed.

We also removed dead code that was not used anywhere except it's spec that were related to Category:Metrics

Screenshots or screen recordings

NA

How to set up and validate locally

Everything else should just work and pipeline should be 💚

  1. Clusters controller should work and open. Head to http://127.0.0.1:3000/<any_project_path>/-/clusters/
  2. Gitlab Prometheus functionality should work as expected. We can rely on tests for this as well.
  3. Rest of the removal is directly linked with Category:Metrics and has the feature flag dependence in one or the other way, so it is removed.

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 #397138 (closed)

Edited by Rajendra Kadam

Merge request reports