618409 - Provide a standalone builder to handle l10n repack respins

Status: RESOLVED FIXED

(Release Engineering :: General, defect)

Description

[Syed Albiz [:salbiz]]

Attachment: tools side standalone-repacks

Re-spinning specific individual locales during a release is a bit painful with the new chunked builders. The standalone builder should alleviate this. The patch doesn't use schedulers at all, so the builder is kicked off through force build, with required properties (locale, release_config, script_repo_revision) set through the force build form.

Comment 1

[Syed Albiz [:salbiz]]

Attachment: use scriptFactory to add standalone_repacks builder

Comment 2

[Armen [:armenzg]]

Good catch!

Comment 3

[[e-mail] (:bhearsum)]

Comment on attachment 496889 [details] [diff] [review]
tools side standalone-repacks

Per real life conversation, we need to support multiple locales in a single run of this. Likely solution is to be comma separated list of locales, which will need to be unpacked by this wrapper. Also shouldn't need the signed stuff.

Comment 4

[[e-mail] (:bhearsum)]

Comment on attachment 496890 [details] [diff] [review]
use scriptFactory to add standalone_repacks builder

Needs adjusting for the signed stuff.

Comment 5

[Syed Albiz [:salbiz]]

Attachment: tools side standalone-repacks (multiple-locale support)

Adjusted to remove signing option and add support for multiple locales at a time

Comment 6

[Syed Albiz [:salbiz]]

Attachment: use scriptFactory to add standalone_repacks builder (adjust to remove signing option)

Removed signing option

Comment 7

[[e-mail] (:bhearsum)]

Comment on attachment 497975 [details] [diff] [review]
tools side standalone-repacks (multiple-locale support)

Almost; use 'unset IFS' to reset it rather than assigning a space. The default value is actually more complex (space, newline, tab -- I think).

Comment 8

[Syed Albiz [:salbiz]]

Attachment: unset IFS properly

Done. Tests out locally.

Comment 9

[[e-mail] (:bhearsum)]

Comment on attachment 497976 [details] [diff] [review]
use scriptFactory to add standalone_repacks builder (adjust to remove signing option)

Landed on default: changeset:   1319:ab8175be13cb


Syed, can you add docs to the Mercurial Release Automation page for this feature?

Comment 10

[[e-mail] (:bhearsum)]

Comment on attachment 498737 [details] [diff] [review]
unset IFS properly

Landed on default: changeset:   1906:8fa9dacec785

Comment 11

[[e-mail] (:bhearsum)]

Landed cleanly on Friday.