Page MenuHomePhabricator

Migrate GrowthExperiments.UserImpactHandler Module to statslib
Closed, ResolvedPublic3 Estimated Story Points

Description

The Stats library (added in MW 1.41) attempts to better define the interface for generating metrics in MediaWiki. See Manual

As part of ensuring the whole of GrowthExperiments is using the Stats Library(Statslib) we should migrate the highlighted areas in GrowthExperiments.UserImpactHandler

Please follow the guidelines and standards outlined in the provided documentation:

Acceptance Criteria

  • MediaWiki.GrowthExperiments.UserImpactHandler.Cache.*
  • MediaWiki.GrowthExperiments.UserImpactHandler.Cache.$UserImpactHandlerCache.sum
  • MediaWiki.GrowthExperiments.UserImpactHandler.PingLimiterTripped.*
  • MediaWiki.GrowthExperiments.UserImpactHandler.PingLimiterTripped.$UserImpactHandlerPingLimiter.rate
  • MediaWiki.timing.growthExperiments.ComputedUserImpactLookup.getExpensiveUserImpact.p99
  • MediaWiki.timing.growthExperiments.ComputedUserImpactLookup.getExpensiveUserImpact.sample_rate

Deploy and Verify changes

  • Deploy the changes to production.
  • Update the relevant dashboard(s) by replacing the old Graphite metric(s) with the new Prometheus metric(s).
  • Verify that the changes have been successfully implemented(This is nothing complex besides normal functionality and metrics captured are sent to Prometheus are queried from in-use dashboards.)

Event Timeline

Sgs triaged this task as High priority.Nov 18 2024, 3:15 PM
Sgs set the point value for this task to 3.

Change #1101836 had a related patch set uploaded (by Cyndywikime; author: Cyndywikime):

[mediawiki/extensions/GrowthExperiments@master] statslib: migrate UserImpactHandler and ComputedUserImpactLookup metrics

https://gerrit.wikimedia.org/r/1101836

Change #1101836 merged by jenkins-bot:

[mediawiki/extensions/GrowthExperiments@master] statslib: migrate UserImpactHandler and ComputedUserImpactLookup metrics

https://gerrit.wikimedia.org/r/1101836