Closed Bug 1455692 Opened 6 years ago Closed 6 years ago

add support for limiting locales in partner repacks


(Release Engineering :: Release Automation: Other, enhancement)

Not set


(firefox62 fixed)

Tracking Status
firefox62 --- fixed


(Reporter: bhearsum, Assigned: bhearsum)



(2 files, 1 obsolete file)

This isn't required for production, but it's super helpful in staging.
Pretty simple! Just skip any locales not in the list, or do all locales if the list is empty.
Attachment #8969755 - Flags: review?(nthomas)
The mozharness part just lets us passthrough to the partner repack script. The partner transform part gathers all locales found in any partner config, and uses that set as the locales to limit to.

It's notable that this will cause --limit-locale to be passed even for full locale releases, which makes the command line long and kindof ugly. I couldn't come up with an obvious fix this.

This was tested on I did some checks of the candidates dir and dep partner bucket, and I _think_ everything looks correct (but I'm not an expert).
Attachment #8969758 - Flags: review?(nthomas)
Attachment #8969758 - Attachment is patch: true
Attachment #8969758 - Attachment mime type: video/dv → text/plain
Attachment #8969755 - Flags: review?(nthomas) → review+
Comment on attachment 8969758 [details] [diff] [review]
add support for limiting locales in mozharness script + transform

I'm not 100% sold on this, but didn't really come up with any great alternatives.
Attachment #8969758 - Flags: review?(nthomas) → review+
This idea is courtesy of Aki, and it seems to work. In my local task graph diffs, this produces the expected locale limiting when using maple-promote-firefox, and doesn't for mb-promote-firefox.,
Attachment #8969758 - Attachment is obsolete: true
Attachment #8973287 - Flags: review?(nthomas)
Comment on attachment 8973287 [details] [diff] [review]
only limit locales on staging branches

lgtm, thanks!
Attachment #8973287 - Flags: review?(nthomas) → review+
Pushed by
add support for limiting locales in partner repacks. r=nthomas
Attachment #8973287 - Flags: checked-in+
Attachment #8969755 - Flags: checked-in+
Closed: 6 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.