Page MenuHomePhabricator

[REPO][SW][GRAFMIGR] (mw.track) Migrate MediaWiki.wikibase.repo.* to statslib
Closed, ResolvedPublic

Description

Background

Graphite is scheduled to become read-only by March 2023 (Q3-FY2024/2025). The sooner metrics are migrated, the more history and continuity a dashboard will have after the transition. Old data can still be queried from Graphite directly. Consider using "Save as" to create a copy of your Grafana dashboard and rename it to "My Name (old Graphite)" to ease viewing of old data in the future, even after you've updated your dashboard to use Prometheus.

PHP change

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.

JS change

TODO:

GetEntities.php
  • MediaWiki.wikibase.repo.api.getentities.entities
MediaWikiEditEntityFactory.php
  • MediaWiki.wikibase.repo.EditEntity.timing.EditEntity.attemptSave
  • MediaWiki.wikibase.repo.EditEntity.timing.EditFilterHookRunner.run
  • MediaWiki.wikibase.repo.EditEntity.timing.EntityStore.saveEntity
  • MediaWiki.wikibase.repo.EditEntity.timing.*.*.item
NOTE: Requires updates to StatsdSaveTimeRecordingEditEntity.php, StatsdSaveTimeRecordingEntityStore.php and StatsdTimeRecordingEditFilterHookRunner.php
TermFallbackCacheFactory.php and TermFallbackCacheFactoryTest.php
  • MediaWiki.wikibase.repo.formatterCache
NOTE: Requires updates to newStatsdRecordingCache, which is in TermFallbackCacheServiceFactory.php, TermFallbackServiceFactoryTest.php and TermFallbackCacheFactoryTest.php
EntityParserOutputGeneratorFactory.php
  • MediaWiki.wikibase.repo.ParserOutputGenerator.timing.getParserOutput.html.*
NOTE: Requires updates to StatsdTimeRecordingEntityParserOutputGenerator.php
wikibase.ui.entitysearch.js
  • MediaWiki.wikibase.repo.searchbox.footer.submit
Other
  • MediaWiki.wikibase.repo.term_store.*

source code

Event Timeline

Lucas_Werkmeister_WMDE renamed this task from Migrate MediaWiki.wikibase.repo.* to statslib to [REPO][SW] Migrate MediaWiki.wikibase.repo.* to statslib.Aug 15 2024, 12:42 PM
karapayneWMDE renamed this task from [REPO][SW] Migrate MediaWiki.wikibase.repo.* to statslib to [REPO][SW][GRAFMIGR] Migrate MediaWiki.wikibase.repo.* to statslib.Aug 29 2024, 8:12 AM

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

[mediawiki/extensions/Wikibase@master] Migrate MediaWiki.wikibase.repo.term_store.* to statslib

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

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

[mediawiki/extensions/Wikibase@master] Migrate MediaWiki.wikibase.repo.api.getentities.entities.* to statslib

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

Krinkle renamed this task from [REPO][SW][GRAFMIGR] Migrate MediaWiki.wikibase.repo.* to statslib to [REPO][SW][GRAFMIGR] (mw.track) Migrate MediaWiki.wikibase.repo.* to statslib.Dec 19 2024, 10:59 PM
Krinkle updated the task description. (Show Details)

Hi @Lucas_Werkmeister_WMDE ,

I wanted to follow up on the patch I submitted a while ago, as it's been pending review for quite some time. Your input and approval are essential to move forward with this work, and I would greatly appreciate it if you could take a moment to review it.

If there's any issue or reason it's been delayed, please let me know so we can address it. Alternatively, if you're unable to review it, could you suggest someone else who might be able to take it on?

Looking forward to your response.

I’ve replied to that Gerrit change, but I don’t know who’s responsible for the rest of this task (there are more metrics to migrate)… IIUC this was in @AndrewTavis_WMDE’s area?

I'd be happy to help as I can, but then I'm not sure if this falls within the scope of work for T371616: [EPIC][GRAFMIGR] Spruce up Wikidata Grafana Metrics, which is the only epic I have scheduled for Grafana/Graphite/Prometheus related work. Assuming it does, I'd wait for @karapayneWMDE to sign off on further support from my end.

Change #1098611 merged by jenkins-bot:

[mediawiki/extensions/Wikibase@master] Remove unused MediaWiki.wikibase.repo.term_store.* metric

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

Change #1121715 had a related patch set uploaded (by Cwhite; author: Cwhite):

[mediawiki/extensions/Wikibase@master] migrate wikibase.repo.searchbox.footer.submit

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

Change #1121715 merged by jenkins-bot:

[mediawiki/extensions/Wikibase@master] add wikibase.repo.searchbox.footer.submit to prometheus

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

Change #1126096 had a related patch set uploaded (by Andrew McAllister (WMDE); author: Andrew McAllister (WMDE)):

[mediawiki/extensions/Wikibase@master] T359251: Migrate MediaWiki.wikibase.repo.* Bug: T359251

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

Change #1126096 had a related patch set uploaded (by Andrew McAllister (WMDE); author: Andrew McAllister (WMDE)):

[mediawiki/extensions/Wikibase@master] T359251: Migrate MediaWiki.wikibase.repo.*

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

Moved to To-Do on our board temporarily as the focus right now is migrating the needed jobs for T377352 :)

The patch for T359252 was combined into Wikibase/+/1126096, as my assumption is that we can't do a scoped migration of one sub-directory of WIkibase. I'd done this for Wikibase/view, but instances of StatsdDataFactory were being called from what I assume were other parts of Wikibase.

@HasanAkgun_WMDE is taking over this task from here :)

Change #1138379 had a related patch set uploaded (by Lucas Werkmeister (WMDE); author: Lucas Werkmeister (WMDE)):

[mediawiki/extensions/Wikibase@master] EditEntity: Hard-code stats prefixes

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

Change #1138380 had a related patch set uploaded (by Lucas Werkmeister (WMDE); author: Lucas Werkmeister (WMDE)):

[mediawiki/extensions/Wikibase@master] Add strict types to files touched in previous change

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

Change #1126096 merged by jenkins-bot:

[mediawiki/extensions/Wikibase@master] Migrate MediaWiki.wikibase.* stats

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

Change #1138379 merged by jenkins-bot:

[mediawiki/extensions/Wikibase@master] EditEntity: Hard-code stats prefixes

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

Change #1138380 merged by jenkins-bot:

[mediawiki/extensions/Wikibase@master] Add strict types to files touched in previous change

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

Change #1139426 had a related patch set uploaded (by Lucas Werkmeister (WMDE); author: Andrew McAllister (WMDE)):

[mediawiki/extensions/Wikibase@wmf/1.44.0-wmf.25] Migrate MediaWiki.wikibase.* stats

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

Change #1139426 merged by jenkins-bot:

[mediawiki/extensions/Wikibase@wmf/1.44.0-wmf.25] Migrate MediaWiki.wikibase.* stats

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

Mentioned in SAL (#wikimedia-operations) [2025-04-28T13:19:55Z] <lucaswerkmeister-wmde@deploy1003> Started scap sync-world: Backport for [[gerrit:1139426|Migrate MediaWiki.wikibase.* stats (T359251 T359252)]], [[gerrit:1139427|Migrate MediaWiki.$prefix.wikibase.client.scribunto.* to statslib (T359253)]]

Mentioned in SAL (#wikimedia-operations) [2025-04-28T13:24:18Z] <lucaswerkmeister-wmde@deploy1003> lucaswerkmeister-wmde: Backport for [[gerrit:1139426|Migrate MediaWiki.wikibase.* stats (T359251 T359252)]], [[gerrit:1139427|Migrate MediaWiki.$prefix.wikibase.client.scribunto.* to statslib (T359253)]] synced to the testservers (https://wikitech.wikimedia.org/wiki/Mwdebug)

Mentioned in SAL (#wikimedia-operations) [2025-04-28T13:33:48Z] <lucaswerkmeister-wmde@deploy1003> Finished scap sync-world: Backport for [[gerrit:1139426|Migrate MediaWiki.wikibase.* stats (T359251 T359252)]], [[gerrit:1139427|Migrate MediaWiki.$prefix.wikibase.client.scribunto.* to statslib (T359253)]] (duration: 13m 52s)