Page MenuHomePhabricator

Remove wgWelcomeSurveyEnableWithHomepage from GrowthExperiments
Closed, ResolvedPublic

Description

Background

The wgWelcomeSurveyEnableWithHomepage configuration value was introduced for English Wikipedia, to ensure Welcome survey only shows to users who will also become a part of the Newcomer Homepage experiment (T284257). This variable was useful back when English Wikipedia was on few percentages of Growth users (for the rest, Welcome survey provided little to none benefits. This has changed, and English Wikipedia now has Growth features enabled for 100% features. This means the variable doesn't bring anything useful to English Wikipedia as of now, and is enabled largely for legacy reasons.

wgWelcomeSurveyEnableWithHomepage works by ignoring Welcome survey experiment settings, and enforcing the default Welcome survey group if the user is selected for the Homepage experiment. This means the changes implemened in T342353 silently didn't take effect on English Wikipedia.

Since we do not plan to experiment with very low percentages for the Newcomer Homepage, I think we can simply remove the support for wgWelcomeSurveyEnableWithHomepage from GrowthExperiments.

Checklist

  • Remove wgWelcomeSurveyEnableWithHomepage from operations/mediawiki-config and thus set it to false (the default) everywhere
  • If there are no apparent issues, remove support for wgWelcomeSurveyEnableWithHomepage from the extension.

Event Timeline

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

[operations/mediawiki-config@master] Growth: Remove wgWelcomeSurveyEnableWithHomepage

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

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

[mediawiki/extensions/GrowthExperiments@master] Remove support for wgWelcomeSurveyEnableWithHomepage

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

Change 951151 merged by jenkins-bot:

[operations/mediawiki-config@master] Growth: Remove wgWelcomeSurveyEnableWithHomepage

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

Mentioned in SAL (#wikimedia-operations) [2023-08-21T20:21:27Z] <urbanecm@deploy1002> Started scap: Backport for [[gerrit:951151|Growth: Remove wgWelcomeSurveyEnableWithHomepage (T342353 T344619)]], [[gerrit:950812|revalidateLinkRecommendations: Load scoreLessThan correctly (T316079)]], [[gerrit:950813|LinkRecommendationUpdater: Load link-recommendation even if disabled (T344343)]]

Mentioned in SAL (#wikimedia-operations) [2023-08-21T20:23:01Z] <urbanecm@deploy1002> urbanecm: Backport for [[gerrit:951151|Growth: Remove wgWelcomeSurveyEnableWithHomepage (T342353 T344619)]], [[gerrit:950812|revalidateLinkRecommendations: Load scoreLessThan correctly (T316079)]], [[gerrit:950813|LinkRecommendationUpdater: Load link-recommendation even if disabled (T344343)]] synced to the testservers mwdebug2001.codfw.wmnet, mwdebug1002.eqiad.wmnet, mwdebug1001.eqiad.wmnet, mwde

Mentioned in SAL (#wikimedia-operations) [2023-08-21T20:32:29Z] <urbanecm@deploy1002> Finished scap: Backport for [[gerrit:951151|Growth: Remove wgWelcomeSurveyEnableWithHomepage (T342353 T344619)]], [[gerrit:950812|revalidateLinkRecommendations: Load scoreLessThan correctly (T316079)]], [[gerrit:950813|LinkRecommendationUpdater: Load link-recommendation even if disabled (T344343)]] (duration: 11m 02s)

I don't see any issues in production enwiki; we should be OK to go ahead with https://gerrit.wikimedia.org/r/c/mediawiki/extensions/GrowthExperiments/+/951153/ now. Moving to CR.

Change 951153 merged by jenkins-bot:

[mediawiki/extensions/GrowthExperiments@master] Remove support for wgWelcomeSurveyEnableWithHomepage

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