1304747 - Redirect existing Belarusian (be) users to Russian (ru) builds via update snippets

Status: RESOLVED FIXED

(Release Engineering :: Release Requests, defect)

Description

[Francesco Lodolo [:flod]]

We plan to stop building Beta and Release for Belarusian (be). 

The agreement with BD is to move the existing users to Russian (ru) through the update system. We don't need to build a special repackage for these users, just to move them to a full Russian build.

Comment 1

[Francesco Lodolo [:flod]]

Attachment: patch for mozilla-aurora

This patch removes be from aurora builds (all-locales), and beta/release (shipped-locales).

Comment 2

[Francesco Lodolo [:flod]]

Comment on attachment 8802903 [details] [diff] [review]
patch for mozilla-aurora

Sorry, clearly wrong bug :-\

Comment 3

[Mihai Tabara [:mtabara]⌚️GMT]

Adding this back to the pool.

Comment 4

[Rail Aliiev [:rail]]

Francesco, do we need to show a What's New Page for these users?

Comment 5

[Rail Aliiev [:rail]]

Attachment: Firefox-51.0b7-build1-Redirect-be-to-ru.json

I created a release blob based on latest beta (51.0b7):

* removed all locales, except "ru"
* removed all "partials"
* removed all channels except "*"
* s/%LOCALE%/ru
* replaced locales.ru with locales.be
* created a new rule on beta-cdntest to match "be" and serve this update

I installed one of the 50 betas in a win64 VM, changed the channel to "beta-cdntest" and updated it to "ru" as expected. If someone can verify this on other platforms, we can move this rule to the beta channel.

Comment 6

[Francesco Lodolo [:flod]]

(In reply to Rail Aliiev [:rail] ⌚️ET from comment #4)
> Francesco, do we need to show a What's New Page for these users?

I'd say no, given that we don't have such a page available on mozilla.org, nor a community to localize it in Belarusian.

Comment 7

[Rail Aliiev [:rail]]

Francesco, I can enable this on the bet channel anytime. How hard do you wan to QA this? Is QA I made in comment 5 sufficient?

Comment 8

[Francesco Lodolo [:flod]]

(In reply to Rail Aliiev [:rail] ⌚️ET from comment #7)
> Francesco, I can enable this on the bet channel anytime. How hard do you wan
> to QA this? Is QA I made in comment 5 sufficient?

Can you explain in details how I can test it (in particular how to set the update channel to "beta-cdntest")? I might give it a try just to check.

Comment 9

[Rail Aliiev [:rail]]

This is what I did:

* download one of the older "be" installers, let's say 50.0b6
* disconnect the VM from the network to prevent accidental background updates
* install Firefox
* in about:config change "app.update.log" to true to log update URLs etc in case I want to debug something (not usually required)
* close Firefox
* edit "defaults\pref\channel-prefs.js" in the app directory (c:\Program Files\...). s/beta/beta-cdntest/.
* Launch Firefox
* Open the console (Ctrl+J IIRC) to see the debug  messages (not usually required)
* Enable networking
* Check for updates, apply the update, restart Firefox.
* Check the UI, maybe omni.ja for locale settings.

I hope this helps.

Comment 10

[Francesco Lodolo [:flod]]

Thanks. I did a test on Linux and Mac, starting from a random 50 beta, and it works as expected. Updates to a Russian build, general.useragent.locale is set to ru instead of be, search settings are from Russian version.

I think we can move on. I'll try to keep a look on Telemetry to see if users actually move away from Belarusian.

Comment 11

[Rail Aliiev [:rail]]

This is live now. I'll keep the bug around to do the same for the release channel in 51.x.

Comment 12

[Rail Aliiev [:rail]]

This is live on the release-cdntest channel now. I repeted the steps in comment 9 using 50.0.2 be and it worked fine. I'll move this rule to the release channel after we release.

Comment 13

[Rail Aliiev [:rail]]

This is live on the release channel now at 25% (to match the main rule). I tested it in a VM and updated from 50.0.2 be to 51.0.1 ru. I'll set it to 100% when we unthrottle 51.0.1

Comment 14

[Rail Aliiev [:rail]]

100% now

Comment 15

[Rail Aliiev [:rail]]

I deleted the rule to simplify the current rules.