Description
Details
Subject | Repo | Branch | Lines +/- | |
---|---|---|---|---|
Removing Gather from enwiki and miscellaneous cosmetic changes | operations/mediawiki-config | master | +2 -5 |
Revisions and Commits
Status | Subtype | Assigned | Task | ||
---|---|---|---|---|---|
Open | None | T106123 Extensions needing to be removed from Wikimedia wikis | |||
Resolved | None | T127509 Uninstall the Gather extension from en.wikipedia.org | |||
Resolved | Tgr | T127784 Notify enwiki Gather users about pending removal of the feature |
Event Timeline
Change 271932 had a related patch set uploaded (by MarcoAurelio):
Removing Gather from enwiki and miscellaneous cosmetic changes
The request is to make it so that neither registered users nor anonymous users can interact with the functionality the extension provides. The simplest way to do this, is to simply removing the extension from the setup as far as I know. In that case, all data will remain in the database, making it possible to migrate or extract at a later time.
The only risk I see here is with any log actions in the public log mechanism. I'm not sure if there are any, but such log entries have proven to be difficult with removing extensions in the past (AFTv5 ?).
Thanks :D
Removing it from enwiki is how I read the RFC and how I see it closed by @TheDJ. I think we can move forward in the next day or so with the removal...I believe @Tnegrin was mentioning wanting to make an at least cursory announcement so those that are using it (however few) aren't caught by surprise.
This doesn't really matter.
MariaDB [enwiki_p]> select count(*) from logging where log_type = 'articlefeedbackv5'; +----------+ | count(*) | +----------+ | 497197 | +----------+ 1 row in set (0.72 sec)
Log and database cleanup (including dumps) can be handled later. Feel free to immediately file reports for such cleanup.
Per discussion with the Reading team, this is blocked on T127784. The target date for uninstalling Gather is next Tuesday (PDT).
Gather uses a ManualLogEntry so uninstalling should not cause any problem with displaying the logs, right?
The i18n message will go away, so it does cause display problems. But given past precedent, it's not a problem worth stopping removal over.
I don't think so, because the message key would be the same as the one in the Gather extension, and I don't think TWN can handle the same key in two extensions?
We can definitely create it locally on en.wp, which is probably good enough in this case.
I don't think TWN can handle the same key in two extensions?
It can actually, just set a custom message prefix https://git.wikimedia.org/blob/translatewiki.git/master/groups%2FMediaWiki%2Fmediawiki-extensions.txt#L2735
ok -- we have the text of the message. Can someone please tell me how to
send it?
-Toby
gather-checkuser-log-action is the message mentioned by @JEumerus, and that comes from ApiEditList::getGatherLogFormattedString (with action being the mode parameter of the list edit API), which is registered as the log handler for gather/action (which is the only log type for Gather as far as I can see). I'm not familiar with the logging system, but just creating that message locally is clearly not going to help. Is there a fallback message key that gets constructed when no handler is registered?
T128056 is not a blocker of this bug, but only of the (future) request to drop the tables.
There are no public logs related to Gather on enwiki, so it shouldn't be a blocker. FYI, T126123 is about this.
Change 271932 merged by jenkins-bot:
Removing Gather from enwiki and miscellaneous cosmetic changes
Done. Log looks broken (if I read the code correctly, with no registered log handler it should fall back to LegacyLogFormatter, which should be able to make more sense of it than this), but that should be a separate task.
I filed T130470 just now to do a postmortem regarding the Gather deployment and subsequent disablement.