Page MenuHomePhabricator

Flagged revs will break if the dimension is set to something other than "accuracy" (except on Wikimedia production)
Open, Needs TriagePublic

Description

It will bring down the whole wiki. See https://gerrit.wikimedia.org/r/c/mediawiki/extensions/FlaggedRevs/+/672570/8/backend/FlaggedRevs.php@88

We need to change the merge strategy.

Event Timeline

Ladsgroup triaged this task as Unbreak Now! priority.Mar 24 2021, 6:59 AM
Ladsgroup created this task.

Change 674485 had a related patch set uploaded (by Ladsgroup; owner: Ladsgroup):
[mediawiki/extensions/FlaggedRevs@master] Make merge_strategy of dimensions to be replace

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

I couldn't reproduce it locally so it's very likely won't break anything but better safe than sorry. ^

Ladsgroup lowered the priority of this task from Unbreak Now! to Needs Triage.Mar 24 2021, 7:29 AM

I'm sure this won't affect production as we rewrite global variables with extension functions, not localsettings but needs fixing regardless. The above patch wouldn't fix it ^

Jdforrester-WMF renamed this task from Flagged revs will break if the dimension is set to something other than "accuracy" to Flagged revs will break if the dimension is set to something other than "accuracy" (except on Wikimedia production).Mar 27 2021, 1:08 AM
Jdforrester-WMF changed the subtype of this task from "Production Error" to "Task".
Jdforrester-WMF updated the task description. (Show Details)

The underling problem here is the fact that extension registry doesn't handle complex array configs (as it shouldn't) one way to move forward here is to replace those arrays with one or two config variables once we dropped the quality tier.

Change 674485 abandoned by Ladsgroup:
[mediawiki/extensions/FlaggedRevs@master] Make merge_strategy of dimensions to be replace

Reason:
The whole config should just be replaced.

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