Page MenuHomePhabricator

[Spike] Figure out what's involved in turning MachineVision off
Closed, ResolvedPublic

Description

Many community members have been unhappy with the MachineVision extension since the start.

Initial work on T339902 suggests that we might need to consider turning it off, this ticket is to make sure we know exactly what would be involved implementation-wise

Related Objects

Event Timeline

Ok here's a suggested timeline for removal:

    • Update the current extension so that the user knows it has been removed. This will involve adding code to return a warning page saying "sorry this extension has been turned off" for Special:SuggestedTags, and returning 410s with a similar msg from the api endpoints (done)
    • Update config to set wgMachineVisionShowUploadWizardCallToAction to false (done)
    • Remove (or mark as read?) all MachineVision-related Echo notifications https://phabricator.wikimedia.org/T347968
  1. make sure all logs/msgs can be displayed ok without the extension https://phabricator.wikimedia.org/T347966
  2. remove all references to machine vision from everywhere else (uploadwizard, dumps, translatewiki, the Isa tool, ) https://phabricator.wikimedia.org/T347967
  3. after X time remove the extension from production (via releng/release) https://phabricator.wikimedia.org/T347972
  4. revert the code with the warnings/410 in the MachineVision repo https://phabricator.wikimedia.org/T347971
  5. archive the database tables, and remove them from production https://phabricator.wikimedia.org/T347969
  6. find out if we also need to archive the event schemas, and if we do then do it https://phabricator.wikimedia.org/T347970

Just for reference - here's a codesearch for suggestedtags, and one for machinevision

Change 955966 had a related patch set uploaded (by Cparle; author: Cparle):

[mediawiki/extensions/MachineVision@master] Disable extension without turning it off

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

Change 955967 had a related patch set uploaded (by Cparle; author: Cparle):

[operations/mediawiki-config@master] Disable UploadWizard CTA for MachineVision

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

Change 955966 merged by jenkins-bot:

[mediawiki/extensions/MachineVision@master] Disable extension without uninstalling it

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

Change 955967 merged by jenkins-bot:

[operations/mediawiki-config@master] Disable UploadWizard CTA for MachineVision

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

Mentioned in SAL (#wikimedia-operations) [2023-09-18T13:04:42Z] <taavi@deploy1002> Started scap: Backport for [[gerrit:955967|Disable UploadWizard CTA for MachineVision (T345187)]]

Mentioned in SAL (#wikimedia-operations) [2023-09-18T13:06:07Z] <taavi@deploy1002> taavi and cparle: Backport for [[gerrit:955967|Disable UploadWizard CTA for MachineVision (T345187)]] synced to the testservers mwdebug1002.eqiad.wmnet, mwdebug1001.eqiad.wmnet, mwdebug2001.codfw.wmnet, mwdebug2002.codfw.wmnet, and mw-debug kubernetes deployment (accessible via k8s-experimental XWD option)

Mentioned in SAL (#wikimedia-operations) [2023-09-18T13:15:59Z] <taavi@deploy1002> Finished scap: Backport for [[gerrit:955967|Disable UploadWizard CTA for MachineVision (T345187)]] (duration: 11m 16s)

Is there a plan to remove the MachineVision extension from Wikimedia sites completely? Or will some of its parts remain enabled?

Planning to remove it completely from commons @Amire80 - afaik that's the only place it's enabled. We're going to start off with a notice that it's disabled, then a while later actually undeploy it

Change 958492 had a related patch set uploaded (by Amire80; author: Amire80):

[translatewiki@master] Move MachineVision from Wikimedia - Media to Legacy

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

Planning to remove it completely from commons @Amire80 - afaik that's the only place it's enabled. We're going to start off with a notice that it's disabled, then a while later actually undeploy it

Thanks! https://gerrit.wikimedia.org/r/958492 defines it as a "Legacy" extension on translatewiki, which is supposed to tell translators that it's the lowest priority.

Change 958492 merged by jenkins-bot:

[translatewiki@master] Move MachineVision from Wikimedia - Media to Legacy

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