Page MenuHomePhabricator

(mw.track) Migrate MediaWiki.cx.publish.*.sum to statslib
Closed, ResolvedPublic

Description

Follow the migration process as outlined below.

Secure/Conduct code review(s).
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 relevant dashboard(s) by replacing the old Graphite metric(s) with the new Prometheus metric(s).
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.

  • MediaWiki.cx.publish.*.sum

Event Timeline

colewhite renamed this task from Migrate MediaWiki.cx.publish.*.sum to statslib to (mw.track) Migrate MediaWiki.cx.publish.*.sum to statslib.Apr 4 2024, 4:25 PM
andrea.denisse changed the task status from Open to In Progress.Nov 19 2024, 7:10 PM
andrea.denisse claimed this task.

Hi @colewhite , I think this task is a duplicate of T359238, what do you think?

Hi @colewhite , I think this task is a duplicate of T359238, what do you think?

I think this can be closed as it does not block us. We have MediaWiki.cx.publish.[fail|success] metrics in Prometheus.

Hi @colewhite,

Should we remove the metrics from MWTrack?

Reference: MWTrack usage in ContentTranslation

Let me know your thoughts.

Hi @colewhite,

Should we remove the metrics from MWTrack?

Reference: MWTrack usage in ContentTranslation

Let me know your thoughts.

Thanks for raising this! It appears I assumed something that wasn't true with my previous comment.

I dug around and found this dashboard still using these metrics. These will need migrated for the top two panels.

Change #1123016 had a related patch set uploaded (by Andrea Denisse; author: Andrea Denisse):

[mediawiki/extensions/ContentTranslation@master] mediawiki.cx: Migrate publish success|fail counter to Prometheus

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

Change #1123030 had a related patch set uploaded (by Andrea Denisse; author: Andrea Denisse):

[mediawiki/extensions/ContentTranslation@master] mediawiki.cx: Migrate publish, save, restore counters to Prometheus

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

Change #1123030 merged by jenkins-bot:

[mediawiki/extensions/ContentTranslation@master] mediawiki.cx: Migrate publish, save, restore counters to Prometheus

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

Change #1123016 merged by jenkins-bot:

[mediawiki/extensions/ContentTranslation@master] mediawiki.cx: Migrate publish success|fail counter to Prometheus

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