Migrate FlaggedRevs periodic mediawiki jobs from mwmaint to mw-cron on kubernetes.
| Job name | Criticality | Done? |
| mediawiki_job_update_flaggedrev_stats.timer | M | X |
ServiceOps new will handle migrating the jobs, but would appreciate input from FlaggedRevs on:
| Status | Subtype | Assigned | Task | ||
|---|---|---|---|---|---|
| Resolved | dancy | T392693 Drop support for Debian Buster | |||
| Resolved | Clement_Goubert | T341560 Migrate mwmaint server functionality to mw-on-k8s | |||
| Resolved | Clement_Goubert | T341555 Implement periodic maintenance scripts for mw-on-k8s | |||
| Resolved | Scott_French | T388535 Migrate flaggedrevs jobs to mw-cron |
Change #1135753 had a related patch set uploaded (by Clément Goubert; author: Clément Goubert):
[operations/puppet@production] alertmanager: Add team/project receivers for Phab
Change #1135754 had a related patch set uploaded (by Clément Goubert; author: Clément Goubert):
[operations/puppet@production] alertmanager: Add routing for task alerts
Change #1135753 merged by Clément Goubert:
[operations/puppet@production] alertmanager: Add team/project receivers for Phab
Change #1135754 merged by Clément Goubert:
[operations/puppet@production] alertmanager: Add routing for task alerts
From a quick read of [0] and (more importantly) [1], this looks relatively low-risk to migrate.
Although there's a lot going on there, the logic in FlaggedRevsStats does not appear to persist (with one caveat, see below) intermediate data or reuse its own outputs written to the flaggedrevs_statistics table - i.e., in the unlikely event that the script produces bad data on k8s / PHP 8.1, we should be able to revert and re-run to fix.
There is one spot in getEditReviewTimes where total edit counts for the respective user type are cached. However, the risk of that query callback suddenly returning incorrect values in this environment seems low.
Change #1144692 had a related patch set uploaded (by Scott French; author: Scott French):
[operations/puppet@production] P:mw:maint:update_flaggedrev_stats: migrate to mw-cron
Change #1144692 merged by Scott French:
[operations/puppet@production] P:mw:maint:update_flaggedrev_stats: migrate to mw-cron
The update-flaggedrev-stats job has been migrated. I'll hold onto this until I'm able to confirm the first run is successful later today.
The first run at 00:08 today (May 14th) completed without issue, and with very similar total elapsed time to the bare-metal case (a bit more than 6m). I've also spot-checked Special:ValidationStatistics on a handful of wikis, and the new stats values look reasonable relative to the prior ones (generated by the update-special-pages runs earlier today).