Skip to content

Change the schema of common_metrics.yml

Reuben Pereira requested to merge 60383-change-common-metrics-schema into master

What does this MR do?

This MR changes the schema of the common_metrics.yml file to the new dashboard yml format described in https://gitlab.com/gitlab-org/gitlab-ce/issues/60383

This change doesn't affect the frontend since the file is not used directly. The metrics described in common_metrics.yml are imported into the DB by db/importers/common_metrics_importer.rb. The APIs used by the frontend read the metrics from the DB (PrometheusMetric model).

EE port: https://gitlab.com/gitlab-org/gitlab-ee/merge_requests/11049

Does this MR meet the acceptance criteria?

Conformity

Performance 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

Closes #60383 (closed)

Edited by Reuben Pereira

Merge request reports