Skip to content

Add an every metric definition CE test

Michał Wielich requested to merge michold_every_metric_ce into master

What does this MR do and why?

Related to #342711 (closed)

We want to make sure that CE Service Ping includes only the keys for metrics that are defined in CE namespace. So far, we only had a similar test running on EE, so it wasn't able to verify the CE keys accuracy.

Screenshots or screen recordings

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

Before After

How to set up and validate locally

  1. Add a change that breaks the CE Usage Ping to metrics definition parity: for example, move a CE metric definition into the EE namespace or add a new metric inside usage_data.rb
  2. Try running the new test with the FOSS_ONLY=1 env variable set. It should fail now.

I have already tested these changes here: !122491 (closed), the failing job

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 Michał Wielich

Merge request reports