Skip to content

Remove GMAU performance_indicator_type from several metric definitions

Carolyn Braza requested to merge cbraza-master-patch-19483 into master

What does this MR do and why?

Describe in detail what your merge request does and why.

This MR fixes removes gmau and paid_gmau as the performance_indicator_types in the counts.static_site_editor_views, usage_activity_by_stage_monthly.create.action_monthly_active_users_design_management, usage_activity_by_stage_monthly.secure.user_secret_detection_jobs, usage_activity_by_stage_monthly.secure.user_license_management_jobs metric definition files.

  • counts.static_site_editor_views: The metric has been removed (status: removed) and it is an all-time counter, so it never should have had those flags in the first place. This will help clean up our data models and prevent create:editor from having multiple metrics designed for GMAU and paid GMAU.
  • usage_activity_by_stage_monthly.create.action_monthly_active_users_design_management: plan:product planning had 2 metrics flagged for paid GMAU. It was agreed in this issue to remove the paid GMAU flag from this metric
  • usage_activity_by_stage_monthly.secure.user_secret_detection_jobs: secure:static analysis had 2 metrics flagged for GMAU and paid GMAU. It was agreed in this issue to remove the GMAU and paid GMAU flags from this metric
  • usage_activity_by_stage_monthly.secure.user_license_management_jobs: secure:composition analysis had 2 metrics flagged for GMAU and paid GMAU. It was agreed in this issue to remove the GMAU and paid GMAU flags from this metric

Screenshots or screen recordings

These are strongly recommended to assist reviewers and reduce the time to merge your change.

How to set up and validate locally

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

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 Carolyn Braza

Merge request reports