Page MenuHomePhabricator

evaluate and migrate in-use parsoid metrics to statslib
Open, Needs TriagePublic

Description

For the following paranoid metrics:

  • MediaWiki.Parsoid.html2wt.setup (html2wt_setup_seconds)
  • MediaWiki.Parsoid.html2wt.selser.serialize (html2wt_serialize_seconds)
  • MediaWiki.Parsoid.html2wt.selser.domDiff (html2wt_domDiff_seconds)
  • MediaWiki.Parsoid.html2wt.init (deprecated May 23)
  • MediaWiki.Parsoid.html2wt.total (deprecated May23)
  • MediaWiki.Parsoid.html2wt.timePerInputKB (deprecated May 23)

Please follow the migration process as outlined below:

  • Secure/Conduct a code review.
  • Deploy the changes to production via the train (https://wikitech.wikimedia.org/wiki/Deployments/Train).
  • Verify that the changes have been successfully implemented.
  • Update the dashboard by replacing the old Graphite metric with the new Prometheus metric.

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

https://www.mediawiki.org/wiki/Manual:Stats for detailed guidance on the conversion process.
https://drive.google.com/file/d/12yQEuOapkML1vb9MgCaX1QzbLBdXE6X2/view for a video tutorial on the conversion process.
https://docs.google.com/presentation/d/1SZWf_D3mWNX-XHN8PHYI84LDZr6GUQC2AMhZ9mQXCI0/edit#slide=id.g2795460c956_0_23 for slides on the best practices for converting metrics to statslib.

Event Timeline

lmata updated the task description. (Show Details)
lmata renamed this task from migrate MediaWiki.Parsoid.html2wt.setup to statslib to migrate top used parsoid to statslib.Jan 11 2024, 9:41 PM
lmata updated the task description. (Show Details)
lmata updated the task description. (Show Details)
lmata renamed this task from migrate top used parsoid to statslib to migrate top used parsoid metrics to statslib.Apr 11 2024, 4:15 PM
lmata renamed this task from migrate top used parsoid metrics to statslib to evaluate and migrate in-use parsoid metrics to statslib.

Change #1052308 had a related patch set uploaded (by Jgiannelos; author: Arlolra):

[mediawiki/services/parsoid@master] Make Utils\Timing helper aware of Wikimedia\Stats lib

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

Change #1053267 had a related patch set uploaded (by Jgiannelos; author: Jgiannelos):

[mediawiki/core@master] Abstract StatsFactory metrics in parsoid SiteConfig

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

Change #1053267 merged by jenkins-bot:

[mediawiki/core@master] Extract StatsFactory methods in parsoid SiteConfig

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

Change #1059421 had a related patch set uploaded (by C. Scott Ananian; author: C. Scott Ananian):

[mediawiki/core@master] Parsoid SiteConfig: allow stat counters incremented by arbitrary values

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

Change #1052308 abandoned by Jgiannelos:

[mediawiki/services/parsoid@master] Make Utils\Timing helper aware of Wikimedia\Stats lib

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

Change #1062375 had a related patch set uploaded (by Jgiannelos; author: Jgiannelos):

[mediawiki/services/parsoid@master] Use statslib for metrics emitted by parsoid

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

Change #1059421 merged by jenkins-bot:

[mediawiki/core@master] Parsoid SiteConfig: allow stat counters incremented by arbitrary values

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

Change #1062375 merged by jenkins-bot:

[mediawiki/services/parsoid@master] Use statslib for metrics emitted by parsoid

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

Change #1071697 had a related patch set uploaded (by C. Scott Ananian; author: C. Scott Ananian):

[mediawiki/vendor@master] Bump wikimedia/parsoid to 0.20.0-a20

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

Change #1071697 merged by jenkins-bot:

[mediawiki/vendor@master] Bump wikimedia/parsoid to 0.20.0-a20

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

Change #1074967 had a related patch set uploaded (by Jgiannelos; author: Jgiannelos):

[mediawiki/services/parsoid@master] stats: Use base units as metrics suffix

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

Change #1074967 merged by jenkins-bot:

[mediawiki/services/parsoid@master] stats: Use units as metrics suffix

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

Change #1078448 had a related patch set uploaded (by Arlolra; author: Arlolra):

[mediawiki/vendor@master] Bump wikimedia/parsoid to 0.20.0-a24

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

Change #1078448 merged by jenkins-bot:

[mediawiki/vendor@master] Bump wikimedia/parsoid to 0.20.0-a24

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

tappof updated the task description. (Show Details)

image.png (525×777 px, 42 KB)

image.png (505×666 px, 35 KB)

The 'init', 'total', and 'timePerInputKB' metrics haven't received data since 27/10/2023.

cscott subscribed.

Re-opening this task, as several histogram-based Parsoid metrics are still not recording correct data in Prometheus, which is blocking us from moving away from the Graphite dashboards.