Page MenuHomePhabricator

Migrate GrowthExperiments.AddLink and ImageRecommendation Related Modules to statslib
Open, HighPublic3 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.AddLink and the ImageRecommendation Related Modules

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

Acceptance Criteria
Update Affected Areas

  • MediaWiki.*.GrowthExperiments.AddLink.growthexperiments_addlink_notinstore.count
  • MediaWiki.GrowthExperiments.CacheBackedImageRecommendationProvider.miss.sum
  • MediaWiki.timing.growthExperiments.imageRecommendationProvider.get.sample_rate
  • MediaWiki.timing.growthExperiments.imageRecommendationProvider.*.p75
  • MediaWiki.timing.growthExperiments.imageRecommendationProvider.*.p99
  • MediaWiki.timing.growthExperiments.LocalSearchTaskSuggester.search.p95
  • MediaWiki.timing.growthExperiments.SearchTaskSuggester.filter.p95
  • MediaWiki.timing.growthExperiments.SearchTaskSuggester.suggest.p95
  • MediaWiki.*.growthexperiments.fixable.link_recommendation.search_index.sum
  • MediaWiki.*.growthexperiments.fixable.link_recommendation.db_table.sum
  • MediaWiki.*.growthexperiments.tasktypecount.link_recommendation.mean

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.)

Related Objects

View Standalone Graph
This task is connected to more than 200 other tasks. Only direct parents and subtasks are shown here. Use View Standalone Graph to show more of the graph.

Event Timeline

Sgs triaged this task as High priority.Nov 18 2024, 3:15 PM

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

[mediawiki/extensions/GrowthExperiments@master] statslib(AddImage): migrate counters and timers

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

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

[mediawiki/extensions/GrowthExperiments@master] statslib(TaskSuggester): migrate counters and timers

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

Sgs set the point value for this task to 3.

Change #1100786 abandoned by Sergio Gimeno:

[mediawiki/extensions/GrowthExperiments@master] statslib(TaskSuggester): migrate counters and timers

Reason:

Squashed in I284e0bb782691b3fef04ce2b4ed5c39674bba910

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

Change #1100785 merged by jenkins-bot:

[mediawiki/extensions/GrowthExperiments@master] statslib(AddImage/TaskSuggester): migrate counters and timers

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

  NODES
Note 3
Project 5
Verify 2