Closed Bug 1366075 Opened 8 years ago Closed 8 years ago

Devedition funsize jobs should use "firefox" prefix in "to_mar"

Categories

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

enhancement

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: rail, Assigned: jlorenzo)

References

Details

Attachments

(2 files)

https://github.com/mozilla/releasetasks/blob/21ce74e75801f08ca8b1dc3de25e550d1e6e766e/releasetasks/templates/desktop/l10n.yml.tmpl#L195 uses {{ product }} for the file name, which is "devedition", but repacks generate files with "firefox" prefix. We should either parameterize this or hardcode "firefox" (what just worked fine in staging :) ).
The same applies to FILENAME_TEMPLATE env variables set in enUS and l10n templates
Assignee: nobody → jlorenzo
Comment on attachment 8869495 [details] releasetasks PR Sorry to bother you. Simon realized he doesn't have the right to formally review releasetasks, and I'm not able to override the review requirement.
Attachment #8869495 - Flags: review?(rail)
Comment on attachment 8869958 [details] Bug 1366075 - Devedition funsize jobs should use "firefox" prefix in "to_mar" https://reviewboard.mozilla.org/r/141514/#review145494 ::: buildfarm/release/release-runner.py:511 (Diff revision 1) > "update_verify_channel": branchConfig.get("update_verify_channel", {}).get(release["product"]), > "update_verify_requires_cdn_push": branchConfig.get("update_verify_requires_cdn_push", False), > } > > validate_graph_kwargs(queue, gpg_key_path, **kwargs) > graph = make_task_graph_strict_kwargs(**kwargs) You also need to touch `make_task_graph_strict_kwargs()` to make it accept the new argument and adjust https://dxr.mozilla.org/build-central/source/tools/buildfarm/release/releasetasks_graph_gen.py#36 to match the current kwargs for consistency.
Attachment #8869958 - Flags: review?(rail) → review-
Attachment #8869495 - Flags: review?(rail) → review+
Comment on attachment 8869958 [details] Bug 1366075 - Devedition funsize jobs should use "firefox" prefix in "to_mar" https://reviewboard.mozilla.org/r/141514/#review145514
Attachment #8869958 - Flags: review?(rail) → review+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: