Skip to content

Move Alerting feature to Core

Vitali Tatarintev requested to merge 26391-move-alerting-feature-to-core into master

What does this MR do?

Related to #210542 (closed)
Contributes to #42640 (closed)

  • Moves Settings > Integrations > Prometheus > Alerts to Core
  • Moves projects_with_prometheus_alerts usage data to Core

Usage data looks like a separate part, but I wasn't sure if it's a good idea to split it and move separately.

Moving usage data before the main feature (Prometheus alerts) to Core might not make sense, because users cannot use the feature yet. On the other hand, moving usage data after release, delays collecting the data of that metric.

So I've put them into the same MR.

Screenshots

There changes is available on "Settings -> Integrations -> Promtheus" page with active Prometheus integration.

Screenshot_2020-04-01_10.44.45

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Security

If this MR contains changes to processing or storing of credentials or tokens, authorization and authentication methods and other items described in the security review guidelines:

  • ~~Label as security and @ mention @gitlab-com/gl-security/appsec~~
  • The MR includes necessary changes to maintain consistency between UI, API, email, or other methods
  • Security reports checked/validated by a reviewer from the AppSec team
Edited by Vitali Tatarintev

Merge request reports

Loading