1799684 - Provide a bouncer URL for the latest release language pack

Status: RESOLVED FIXED

(Release Engineering :: Release Automation, enhancement)

Description

[Mike Kaply [:mkaply]]

We have enterprises that want to install language packs via policy, but unfortunately there's no easy way to get the latest release language pack.

On AMO, the latest.xpi URL will point to the latest XPI uploaded which will be beta or nightly.

And on our download websites, you have to put the version in the URL, so you can't have a URL to download the "latest release" or "latest ESR" language pack.

It would be nice to have something like:

https://download.mozilla.org/?product=langpack-latest-ssl&lang=en-US

That could download a languagepack XPI.

Comment 1

[Mike Kaply [:mkaply]]

Any thoughts on how easy this would be to do?

Comment 2

[Julien Cristau [:jcristau]]

This should be reasonably simple, ni?self so I don't forget.

Comment 3

[BMO Automation]

Attachment: [mozilla-releng/scriptworker-scripts] Bug 1799684 - bouncerscript: allow bouncer entries for firefox langpacks (#621)

Comment 4

[Julien Cristau [:jcristau]]

Attachment: Bug 1799684 - add bouncer entries for langpacks

Comment 5

[Julien Cristau [:jcristau]]

Attachment: Bug 1799684 - add bouncer-check support for language packs

Comment 6

[Julien Cristau [:jcristau]]

Attachment: Bug 1799684 - add bouncer-check support for -latest langpack aliases

Comment 7

[Julien Cristau [:jcristau]]

Mike, the patches here would enable URLs like https://download.mozilla.org/?product=Firefox-beta-langpack-latest-SSL&os=osx&lang=fr (and eventually the same with product=Firefox-langpack-latest-SSL and product=Firefox-esr-langpack-latest-SSL), hopefully that'll work for the described use case?

Comment 8

[Mike Kaply [:mkaply]]

This looks perfect.

I only wish we'd stop duplicating language packs for all platforms unnecessarily :)

Comment 9

[BMO Automation]

Comment on attachment 9313546 [details] [review]
[mozilla-releng/scriptworker-scripts] Bug 1799684 - bouncerscript: allow bouncer entries for firefox langpacks (#621)

GitHub pull request attachment was moved to bug 1812378. Setting attachment 9313546 [details] [review] to obsolete.

Comment 10

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/6dbf405aa947
add bouncer entries for langpacks r=releng-reviewers,gbrown

Comment 11

[Sandor Molnar[:smolnar]]

https://hg.mozilla.org/mozilla-central/rev/6dbf405aa947

Comment 12

[Mike Kaply [:mkaply]]

Is there some other change that is needed to make the bouncer URLs work?

Comment 13

[Julien Cristau [:jcristau]]

They should start working as this rides the trains (or we can uplift to beta/esr102 if we want).

Comment 14

[Mike Kaply [:mkaply]]

I would love to uplift this to beta/esr102 if it's low risk.

Comment 15

[Julien Cristau [:jcristau]]

Comment on attachment 9313549 [details]
Bug 1799684 - add bouncer entries for langpacks

Beta/Release Uplift Approval Request

• User impact if declined: enterprises don't have a version-agnostic url to use for language packs, for use in policy settings.
• Is this code covered by automated tests?: No
• Has the fix been verified in Nightly?: No
• Needs manual test from QE?: No
• If yes, steps to reproduce:
• List of other uplifts needed: None
• Risk to taking this patch: Low
• Why is the change risky/not risky? (and alternatives if risky): Adds an extra bouncer product/alias during release automation, pointing at the language packs on archive.mozilla.org. no impact on existing products/URLs/releases.
  There's a bouncer-check test as part of release automation and cron, but we can't land that patch until after a first release creates the URLs. We don't have to advertise the new URLs until after we've double checked them, though.
• String changes made/needed:
• Is Android affected?: No

ESR Uplift Approval Request

• If this is not a sec:{high,crit} bug, please state case for ESR consideration: enterprise-facing improvement
• User impact if declined:
• Fix Landed on Version: 111
• Risk to taking this patch: Low
• Why is the change risky/not risky? (and alternatives if risky):

Comment 16

[Pascal Chevrel:pascalc]

Comment on attachment 9313549 [details]
Bug 1799684 - add bouncer entries for langpacks

Approved for 110 beta 8, thanks.

Comment 17

[Pascal Chevrel:pascalc]

https://hg.mozilla.org/releases/mozilla-beta/rev/a5014b85a7db

Comment 18

[Julien Cristau [:jcristau]]

Attachment: Bug 1799684 - add bouncer-check support for beta langpacks. r?gbrown

Comment 19

[Julien Cristau [:jcristau]]

Attachment: Bug 1799684 - add bouncer-check support for release langpacks. r?gbrown

Comment 20

[Julien Cristau [:jcristau]]

Attachment: Bug 1799684 - add bouncer-check support for esr langpacks. r?gbrown

Comment 21

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/8cd8d77713f4
add bouncer-check support for beta langpacks. r=gbrown DONTBUILD

Comment 22

[Julien Cristau [:jcristau]]

Comment on attachment 9315337 [details]
Bug 1799684 - add bouncer-check support for beta langpacks. r?gbrown

Mind uplifting this a=release? It'll add a check during release promotion and in cron for the new bouncer URLs.

Comment 23

[Sandor Molnar[:smolnar]]

https://hg.mozilla.org/mozilla-central/rev/8cd8d77713f4

Comment 24

[Pascal Chevrel:pascalc]

https://hg.mozilla.org/releases/mozilla-beta/rev/e9c910a8ff48

Comment 25

[Julien Cristau [:jcristau]]

Attachment: Bug 1799684 - bouncer-check: don't look for langpacks on win64-aarch64

win64-aarch64 is not included in beetmover's list of platforms where we copy langpacks.

Comment 26

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/c1b1eb90b879
bouncer-check: don't look for langpacks on win64-aarch64 DONTBUILD

Comment 27

[Julien Cristau [:jcristau]]

https://hg.mozilla.org/releases/mozilla-beta/rev/a8df88a2f38f

Comment 28

[Pascal Chevrel:pascalc]

Julien, the patch doesn't graft cleanly to the esr102 branch as bug 1786291 wasn't uplifted to the ESR102 branch.

Comment 29

[Pascal Chevrel:pascalc]

https://hg.mozilla.org/releases/mozilla-esr102/rev/812b53769ac9

Comment 30

[Iulian Moraru]

https://hg.mozilla.org/mozilla-central/rev/c1b1eb90b879

Comment 31

[Julien Cristau [:jcristau]]

Attachment: Bug 1799684 - create the firefox-esr-next-langpack-latest-ssl bouncer alias with 102.8.0esr.

Creating the bouncer alias now means we don't have to wait until
115.0esr is shipped before adding it to bouncer-check.

This patch should land on esr102 only, and should get backed out after
102.8.0esr ships.

Comment 32

[Julien Cristau [:jcristau]]

https://hg.mozilla.org/releases/mozilla-esr102/rev/3e15a0e2be47

Comment 33

[Julien Cristau [:jcristau]]

Comment on attachment 9315338 [details]
Bug 1799684 - add bouncer-check support for release langpacks. r?gbrown

Beta/Release Uplift Approval Request

• User impact if declined: n/a
• Is this code covered by automated tests?: No
• Has the fix been verified in Nightly?: No
• Needs manual test from QE?: No
• If yes, steps to reproduce:
• List of other uplifts needed: None
• Risk to taking this patch: Low
• Why is the change risky/not risky? (and alternatives if risky): adding new langpack URLs to the daily bouncer checks
• String changes made/needed:
• Is Android affected?: No

Comment 34

[Julien Cristau [:jcristau]]

Comment on attachment 9315339 [details]
Bug 1799684 - add bouncer-check support for esr langpacks. r?gbrown

ESR Uplift Approval Request

• If this is not a sec:{high,crit} bug, please state case for ESR consideration: adding new langpack URLs to daily and per-release bouncer-check so we know if they break
• User impact if declined: none
• Fix Landed on Version: 112
• Risk to taking this patch: Low
• Why is the change risky/not risky? (and alternatives if risky):

Comment 35

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/6e79055edec7
add bouncer-check support for release langpacks. r=gbrown DONTBUILD
https://hg.mozilla.org/integration/autoland/rev/47be2ade69df
add bouncer-check support for esr langpacks. r=gbrown DONTBUILD

Comment 36

[Norisz Fay [:noriszfay]]

https://hg.mozilla.org/mozilla-central/rev/6e79055edec7
https://hg.mozilla.org/mozilla-central/rev/47be2ade69df

Comment 37

[Donal Meehan [:dmeehan]]

Comment on attachment 9315338 [details]
Bug 1799684 - add bouncer-check support for release langpacks. r?gbrown

Approved for 111.0b5

Comment 38

[Donal Meehan [:dmeehan]]

https://hg.mozilla.org/releases/mozilla-beta/rev/ce248640e318

Comment 39

[Pascal Chevrel:pascalc]

Comment on attachment 9315338 [details]
Bug 1799684 - add bouncer-check support for release langpacks. r?gbrown

Approved for 110 branch.

Comment 40

[Donal Meehan [:dmeehan]]

Comment on attachment 9315339 [details]
Bug 1799684 - add bouncer-check support for esr langpacks. r?gbrown

Approved for 102.9esr.

Comment 41

[Donal Meehan [:dmeehan]]

https://hg.mozilla.org/releases/mozilla-esr102/rev/50febcb5a2f4

Comment 42

[Donal Meehan [:dmeehan]]

Backed out changeset 50febcb5a2f4 (Bug 1799684) backed out for introducing test failures
esr102 Backout link

:jcristau could you take a look?
Log: https://firefox-ci-tc.services.mozilla.com/tasks/ZZGRDDUYRQWMWJOyE9ENKQ/runs/0/logs/public/logs/live.log

Comment 43

[Julien Cristau [:jcristau]]

https://hg.mozilla.org/releases/mozilla-esr102/rev/dc581269bb88748d74ab6df89dbef11070ff4d70

Comment 44

[Julien Cristau [:jcristau]]

verified that cron-bouncer-check now passes on esr102: https://firefox-ci-tc.services.mozilla.com/tasks/dMQb0BPWSXCVQRYs3r156Q

Comment 45

[Julien Cristau [:jcristau]]

And backed out the change from comment 32 on esr102 to stop updating firefox-esr-next-langpack-latest-ssl until esr115:
https://hg.mozilla.org/releases/mozilla-esr102/rev/92edc0e8bc7edbd7cae690c82ce374f58e4db138