Page MenuHomePhabricator

Remove MvpImageRecommendationApiHandler from GrowthExperiments
Open, In Progress, LowPublic

Description

MvpImageRecommendationApiHandler is currently configured as the default add an image suggestion handler. It relies on https://image-suggestion-api.wmcloud.org, which is currently down, and that service has very old suggestions plus no one has the intention of maintaining it.

Production makes use of the image-suggestion service (https://wikitech.wikimedia.org/wiki/Image-suggestion). This service is currently available only from the production cluster (making it publicly available is tracked as T306349). This makes the production endpoint unusable for local developing (without setting up a tunel between developer's setup and WMF production).

In https://gerrit.wikimedia.org/r/c/mediawiki/extensions/GrowthExperiments/+/916880, we added ActionApiImageRecommendationApiHandler. This makes it possible for developers to use data loaded from a given remote wiki (such as, a WMF production one). This should be sufficient for local development purposes.

To unblock archival of https://image-suggestion-api.wmcloud.org (T344740 plus related Slack convo), we should switch to ActionApiImageRecommendationApiHandler as the default handler and remove support for the MVP one.

Event Timeline

Change 954620 had a related patch set uploaded (by Urbanecm; author: Urbanecm):

[operations/mediawiki-config@master] beta: Do not reference image-suggestion-api.wmcloud.org

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

Change 954670 had a related patch set uploaded (by Urbanecm; author: Urbanecm):

[mediawiki/extensions/GrowthExperiments@master] WIP: Remove MvpImageRecommendationApiHandler

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

Urbanecm_WMF changed the task status from Open to In Progress.Sep 5 2023, 1:33 PM

Change 954620 merged by jenkins-bot:

[operations/mediawiki-config@master] beta: Do not reference image-suggestion-api.wmcloud.org

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

Change 954670 abandoned by Urbanecm:

[mediawiki/extensions/GrowthExperiments@master] Remove MvpImageRecommendationApiHandler

Reason:

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