Page MenuHomePhabricator

Deploy Phonos to beta cluster
Closed, ResolvedPublic5 Estimated Story Points

Description

https://www.mediawiki.org/wiki/Writing_an_extension_for_deployment#Deploy_to_Beta_Cluster

  • Add the new extension submodule to the git mediawiki/extensions repo if it's not already in it. See example. This will result in the code being deployed (unused) to the Beta Cluster.
  • Move your extension's CI config to the "Wikimedia production" section, add the "in-wikimedia-production" job template, and make sure it has and passes all the expected jobs for production code. See example.
  • Add your extension to the json make-wmf-branch release tool at least two weeks prior to your target date for enabling on the Beta Cluster. It will add the extension as a submodule of mediawiki/core when the weekly deploy branch is cut, and the code will be deployed (unused) to production (see explanation in the next step).
  • Add your extension to extension-list. See example. This requires that the code be present on every branch running in production, since extension-list is used to build the CDB database for i18n files in both Beta and production.
  • Add your extension config variable to InitialiseSettings.php and set it to be default false. See example.
  • Add your extension config variable (same as in previous step) to InitialiseSettings-labs.php and set it to be true on Beta Cluster wikis you want it to be on. You may want to turn it off for loginwiki (which doesn't have most extensions). See example.
  • Load your extension in CommonSettings-labs.php. See https://gerrit.wikimedia.org/r/c/operations/mediawiki-config/+/455601/.
  • Per T316011: Rollout plan for Phonos, enable on Beta en.wiki, en.wikt and en-rtl.wiki

Related Objects

Event Timeline

There are a very large number of changes, so older changes are hidden. Show Older Changes

Change 821249 had a related patch set uploaded (by Samtar; author: Samtar):

[operations/mediawiki-config@master] extension-list: Add Phonos

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

Change 822656 had a related patch set uploaded (by Samtar; author: Samtar):

[operations/mediawiki-config@master] InitialiseSettings: Add wmgUsePhonos (default => false)

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

TheresNoTime changed the task status from Open to Stalled.EditedAug 15 2022, 9:59 PM

Waiting for at least two mw trains per comments at https://gerrit.wikimedia.org/r/c/operations/mediawiki-config/+/821249


T314186: 1.39.0-wmf.25 deployment blockers dependent, should be set to move this patch forwards on Friday 19th

Change 821249 merged by jenkins-bot:

[operations/mediawiki-config@master] extension-list: Add Phonos

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

Mentioned in SAL (#wikimedia-operations) [2022-08-17T20:29:58Z] <samtar@deploy1002> Synchronized wmf-config/extension-list: Config: [[gerrit:821249|extension-list: Add Phonos (T314294)]] (duration: 03m 17s)

Change 822656 merged by jenkins-bot:

[operations/mediawiki-config@master] InitialiseSettings: Add wmgUsePhonos (default => false)

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

Mentioned in SAL (#wikimedia-operations) [2022-08-17T20:36:00Z] <samtar@deploy1002> Synchronized wmf-config/InitialiseSettings.php: Config: [[gerrit:822656|InitialiseSettings: Add wmgUsePhonos (default => false) (T314294)]] (duration: 03m 29s)

Change 824291 had a related patch set uploaded (by Samtar; author: Samtar):

[operations/mediawiki-config@master] InitialiseSettings-labs: Enable Phonos on beta enwiki

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

Going to set Beta to use larynx for initial testing

Change 824294 had a related patch set uploaded (by Samtar; author: Samtar):

[operations/mediawiki-config@master] CommonSettings: Load Phonos extension

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

https://gerrit.wikimedia.org/r/824291 is scheduled for tomorrow's UTC afternoon backport window β€” this will set https://en.wikipedia.beta.wmflabs.org as the only beta cluster wiki to have Phonos enabled. It will not enable the extension.

// T314294
'wmgUsePhonos' => [
    'default' => false,
    'enwiki' => true,
],

Once this is deployed, the remaining patch will enable Phonos β€” we should discuss and agree on if Phonos is stable enough to be enabled.

I've initially added the following configuration:

if ( $wmgUsePhonos ) {
    wfLoadExtension( 'Phonos' );
    // T314294#8163339 β€” Initial testing configuration
    $wgPhonosEngine = 'larynx';
    $wgPhonosApiEndpointLarynx = 'https://larynx-tts.wmcloud.org/api/tts';
}
TheresNoTime changed the task status from Open to Stalled.Aug 18 2022, 11:43 AM

Pending a decision on enable now vs enable after T315482: Generate HTML server side for IPA audio button (and related) β€” either option is is stable enough for deployment imho

Change 824291 merged by jenkins-bot:

[operations/mediawiki-config@master] InitialiseSettings-labs: Enable Phonos on beta enwiki

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

Mentioned in SAL (#wikimedia-operations) [2022-08-18T13:13:33Z] <samtar@deploy1002> Synchronized wmf-config/InitialiseSettings-labs.php: Config: [[gerrit:824291|InitialiseSettings-labs: Enable Phonos on beta enwiki (T314294)]] (duration: 03m 30s)

TheresNoTime changed the task status from Stalled to Open.Sep 5 2022, 8:49 AM

Unstalled in T316020#8206955

Change 824294 merged by jenkins-bot:

[operations/mediawiki-config@master] CommonSettings-labs: Load Phonos extension

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

Mentioned in SAL (#wikimedia-operations) [2022-09-06T13:19:41Z] <samtar@deploy1002> Synchronized wmf-config/CommonSettings-labs.php: Config: [[gerrit:824294|CommonSettings-labs: Load Phonos extension (T314294)]] (duration: 04m 05s)

Change 830567 had a related patch set uploaded (by Samtar; author: Samtar):

[operations/mediawiki-config@master] CommonSettings-labs: Set config to production-esque values

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

Change 830567 merged by jenkins-bot:

[operations/mediawiki-config@master] CommonSettings-labs: Set config to production-esque values

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

Mentioned in SAL (#wikimedia-operations) [2022-09-07T13:55:56Z] <samtar@deploy1002> Synchronized wmf-config/CommonSettings-labs.php: Config: [[gerrit:830567|CommonSettings-labs: Set config to production-esque values (T314294)]] (duration: 03m 47s)

TheresNoTime changed the task status from Open to Stalled.Sep 8 2022, 5:32 PM

Moving to QA but stalling due to T317195: Failed to create storage directory on Beta Cluster β€” once that is resolved, this should be QAable

Will put this in QA, I suppose the acceptance criteria is "Phonos works on the Beta Cluster"

Change 841547 had a related patch set uploaded (by Samtar; author: Samtar):

[operations/mediawiki-config@master] InitialiseSettings-labs: Enable Phonos on en_rtlwiki, enwiktionary

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

Change 841547 merged by jenkins-bot:

[operations/mediawiki-config@master] InitialiseSettings-labs: Enable Phonos on en_rtlwiki, enwiktionary

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

Mentioned in SAL (#wikimedia-operations) [2022-10-11T15:30:59Z] <TheresNoTime> deployed beta cluster only change, [[gerrit:841547]], for T314294

@TheresNoTime Phonos passed on all 3 links but we did want to ask you one more thing before moving it to production sign-off. We noticed that on the first link (https://en.wikipedia.beta.wmflabs.org/wiki/Phonos), a few of the phonos words that are highlighted in the screenshot say the text word instead of the actual word. We are not sure if this is an issue or not but please let us know if this is ok to move it to production sign-off. Thanks!

T314294_PhonosToBeta.png (945Γ—3 px, 342 KB)

Links tested and successfully passed

@TheresNoTime Phonos passed on all 3 links but we did want to ask you one more thing before moving it to production sign-off. We noticed that on the first link (https://en.wikipedia.beta.wmflabs.org/wiki/Phonos), a few of the phonos words that are highlighted in the screenshot say the text word instead of the actual word. We are not sure if this is an issue or not but please let us know if this is ok to move it to production sign-off. Thanks!

T314294_PhonosToBeta.png (945Γ—3 px, 342 KB)

Links tested and successfully passed

Thanks for letting me know β€” I'm going to consider that its own issue (which is tracked elsewhere I believe), and given that Phonos is "working" on the Beta Cluster I'll move this to sign-off :)

Nice work. Someone might want to check if different text sizes can be handled a bit better perhaps ? https://en.wikipedia.beta.wmflabs.org/wiki/Phonos#Text_sizes

{F35564149}

Nice work. Someone might want to check if different text sizes can be handled a bit better perhaps ? https://en.wikipedia.beta.wmflabs.org/wiki/Phonos#Text_sizes

{F35564149}

Thanks for raising :) I've split this report into T320820: Phonos speaker icon placement moves on increased/reduced font size (though I think there may be another task/patch in progress which would resolve this)