1465901 - Expose set of locales to multi-locale builds as TC environment variables

Status: ASSIGNED

(Firefox Build System :: General, enhancement)

Description

[Nick Alexander :nalexander [he/him] (PTO August 22-September 12)]

Right now, the set of locales that get packaged into a (mobile/android) multi-locale build is controlled by configuration files deep in the guts of mozharness 

https://searchfox.org/mozilla-central/source/testing/mozharness/configs/multi_locale

and those all seem to redirect to another JSON file in the tree, e.g.

https://searchfox.org/mozilla-central/source/testing/mozharness/configs/multi_locale/mozilla-central_android.json#5

points to

https://searchfox.org/mozilla-central/source/mobile/locales/l10n-changesets.json

We want to have all of the locale extraction, filtering, etc for multi-locale builds done in the taskgraph configuration, much like it's done for single-locale repacks now.  See

https://dxr.mozilla.org/mozilla-central/source/taskcluster/taskgraph/transforms/l10n.py#323

This ticket tracks exposing the set of locales to the Task Cluster environment directly, so that multi-locale mozconfig files can include them directly.  That would let us move toward the world sketched in

https://bugzilla.mozilla.org/show_bug.cgi?id=1445701#c6

Comment 1

[Phabricator Automation]

Attachment: Bug 1465901 - Expose set of locales to multi-locale builds as TC environment variables.

Comment 2

[Justin Wood (:Callek)]

(In reply to Phabricator Automation from comment #1)
> Created attachment 8982598 [details]
> Bug 1465901 - Expose set of locales to multi-locale builds as TC environment
> variables.

Nick, this should expose it as a variable, I didn't do any mozharness changes with this, nor is the implementation as clean as it could be, but this would make it work. I'm open to better env variables or formats too, this is just what I came up with today.

Comment 3

[Kim Moir [:kmoir] ET]

triaging, assigning to callek since he attached patches

Comment 4

[Nick Alexander :nalexander [he/him] (PTO August 22-September 12)]

(In reply to Justin Wood (:Callek) from comment #2)
> (In reply to Phabricator Automation from comment #1)
> > Created attachment 8982598 [details]
> > Bug 1465901 - Expose set of locales to multi-locale builds as TC environment
> > variables.
> 
> Nick, this should expose it as a variable, I didn't do any mozharness
> changes with this, nor is the implementation as clean as it could be, but
> this would make it work. I'm open to better env variables or formats too,
> this is just what I came up with today.

I've been out of this loop for a long time, so clearing the NI, since I'm not going to act on this any time soon.  Sorry, Callek!

Comment 5

[Nick Alexander :nalexander [he/him] (PTO August 22-September 12)]

I was just about to file basically this ticket, so I'll repurpose it instead.