Page MenuHomePhabricator

Migrate GrowthExperiments.UserVariant, GrowthExperiments.UsersNotOptedIntoGrowthFeatures & GrowthExperiments.UsersOptedIntoGrowthFeatures Module to statslib
Closed, ResolvedPublic

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.UserVariant, GrowthExperiments.UsersNotOptedIntoGrowthFeatures & GrowthExperiments.UsersOptedIntoGrowthFeatures

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

Acceptance Criteria
Update Affected Areas

  • MediaWiki.$wiki.GrowthExperiments.UsersNotOptedIntoGrowthFeatures.sum
  • MediaWiki.$wiki.GrowthExperiments.UsersOptedIntoGrowthFeatures.sum
  • MediaWiki.$wiki.GrowthExperiments.UserVariant.$variant.sum
  • MediaWiki.*.GrowthExperiments.UserVariant.*
  • MediaWiki.*.GrowthExperiments.UserVariant.*.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

Change #1102347 had a related patch set uploaded (by Sergio Gimeno; author: Sergio Gimeno):

[mediawiki/extensions/GrowthExperiments@master] statslib(HomepageHooks): migrate counters

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

Sgs moved this task from Maintenance to Current Sprint on the Growth-Team board.
Sgs moved this task from Incoming to Doing on the Growth-Team (Current Sprint) board.

Change #1102347 merged by jenkins-bot:

[mediawiki/extensions/GrowthExperiments@master] statslib(HomepageHooks): migrate counters

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