1569892 - Allow for Fluent files that are not ready for localization

Status: RESOLVED FIXED

(Core :: Internationalization, defect, P1)

Description

[Axel Hecht [:Pike]]

Our current approach for packaging Fluent that's not ready for l10n doesn't work.

What we did was packaging the pre-release files in the same /localization path as we will in prod.

At repack time, that breaks all kinds of things once you actually have them in localizations, as we see in bugs 1568438, 1569807.

I'm still counting on how many streams we cross, filing this in Intl right now, but I might also move this.

Here's what we need to keep from the current behavior:

• The ftl files from content are packages for all locales, as if they were localized. That's a workaround for bug 1464156.
• We can change the contents in content ftl files freely, w/out having to care about IDs.

I read the latter to mean that we can't use the localized fluent file as a drop-in replacement for a content one, in a cross-channel world. The IDs we have on l10n-central might overlap with different content-content.

Thus, we should have distinct path locations for these files. I propose that we should have use preview/aboutLogins.ftl instead of browser/aboutLogins.ftl.

Now, there's an interesting question around packaging. We can continue to do what we do now, and package the files a few times. Or we could package them once in a known location in omni.ja, and add to browser/l10n-registry.manifest a line like

category l10n-registry 0-preview resource://app/fluent-content/

(and move toolkit to 2).

Given that this would be the last location to look at in regular builds, I don't think this should come with perf problems. Maybe as a follow-up?

Comment 1

[Axel Hecht [:Pike]]

[Tracking Requested - why for this release]:

Requesting tracking for 69 and 68 esr, 'cause this is required to take string updates, see bug 1569807.

Pushed something to try, let's see how that goes: https://treeherder.mozilla.org/#/jobs?repo=try&revision=7817d73d6fdc68430e82de794a7adeec1a3b7f20. That patch won't apply to beta or esr at all, but the concept of it should.

Comment 2

[Axel Hecht [:Pike]]

Attachment: Bug 1569892, move in-progress fluent files to preview locations, r=johannh,mtigley

Having these files in their final packaged locations creates problems when
we expose them to localizations.

Comment 3

[BugBot [:suhaib / :marco/ :calixte]]

Bugbug thinks this bug is a enhancement, but please change it back in case of error.

Comment 4

[Axel Hecht [:Pike]]

Attachment: Bug 1569892, move in-progress fluent files to preview locations, r=jaws

This is the beta variation of D39872.

Comment 5

[Axel Hecht [:Pike]]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=ddb0015f0bede2e6b777cf24177971a91eb1ef46 is trying beta repacks now.

The same patch mostly applies to 68 and 68esr, aside from chunk to browser/components/aboutlogins/AboutLoginsParent.jsm, which isn't needed.

Comment 6

[Axel Hecht [:Pike]]

This is green on beta for linux and windows. There's an unrelated bustage on macos, as that's mismatching Firefox Nightly.app and Firefox.app, that's unrelated to this bug.

Comment 7

[Axel Hecht [:Pike]]

Comment on attachment 9081606 [details]
Bug 1569892, move in-progress fluent files to preview locations, r=jaws

Beta/Release Uplift Approval Request

• User impact if declined: We can't take updates to l10n on beta
• Is this code covered by automated tests?: Yes
• 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): This succeeded on try, and has feedback from releng, l10n, and firefox engineering.
• String changes made/needed: None

Comment 8

[Axel Hecht [:Pike]]

Comment on attachment 9081606 [details]
Bug 1569892, move in-progress fluent files to preview locations, r=jaws

Beta/Release Uplift Approval Request

• User impact if declined: We can't take string updates for 68/esr68.
• Is this code covered by automated tests?: Yes
• 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):
• String changes made/needed:

ESR Uplift Approval Request

• If this is not a sec:{high,crit} bug, please state case for ESR consideration: We can't take string updates for 68/esr68, which we want for a ESR bug.
• User impact if declined: We can't take string updates for 68/esr68.
• Fix Landed on Version:
• Risk to taking this patch: Low
• Why is the change risky/not risky? (and alternatives if risky):
• String or UUID changes made by this patch:

Comment 9

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/0c94da5ac908
move in-progress fluent files to preview locations, r=johannh

Comment 10

[Daniel Varga [:dvarga]]

https://hg.mozilla.org/mozilla-central/rev/0c94da5ac908

Comment 11

[Ryan VanderMeulen [:RyanVM]]

Comment on attachment 9081606 [details]
Bug 1569892, move in-progress fluent files to preview locations, r=jaws

Un-breaks Beta l10n repacks. Approved for 69.0b10.

Comment 12

[Sebastian Hengst [:aryx] (needinfo me if it's about an intermittent or backout)]

https://hg.mozilla.org/releases/mozilla-beta/rev/7b2d5bd73d93

Comment 13

[Ryan VanderMeulen [:RyanVM]]

Comment on attachment 9081606 [details]
Bug 1569892, move in-progress fluent files to preview locations, r=jaws

Today's b10 build went great with this patch. Approving for 68.0.2/68.0.2esr as well.

Comment 14

[Sebastian Hengst [:aryx] (needinfo me if it's about an intermittent or backout)]

https://hg.mozilla.org/releases/mozilla-release/rev/b8459608aee5

Comment 15

[Sebastian Hengst [:aryx] (needinfo me if it's about an intermittent or backout)]

https://hg.mozilla.org/releases/mozilla-esr68/rev/b534799934d1

Comment 16

[Sebastian Hengst [:aryx] (needinfo me if it's about an intermittent or backout)]

FIREFOX_ESR_68_0_X_RELBRANCH https://hg.mozilla.org/releases/mozilla-esr68/rev/63214909da19

Comment 17

[Emma Humphries ☕️🎸🧞‍♀️✨ (she/they) [:emceeaich] (Pacific Time) use needinfo]

This bug has been identified as part of a pilot on determining root causes of blocking and dot release drivers.

It needs a root-cause set for it. Please see the list at https://docs.google.com/document/d/1FFEGsmoU8T0N8R9kk-MXWptOPtXXXRRIe4vQo3_HgMw/.

Add the root cause as a whiteboard tag in the form [rca - <cause> ] and remove the rca-needed keyword.

If you have questions, please contact :tmaity.

Comment 18

[Makoto Kato [:m_kato]]

https://bugzilla.mozilla.org/show_bug.cgi?id=1569892#c0 is description.

Comment 19

[Firefox Bug Husbandry Bot]

Please specify a root cause for this bug. See :tmaity for more information.