1208182 - unstub funsize mar urls in release promotion

Status: RESOLVED FIXED

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

Description

[[e-mail] (:bhearsum)]

Funsize jobs are kicked off now, but they don't work due to bad URLs!

Comment 1

[Rail Aliiev [:rail]]

Attachment: do it!

This also addresses bug 1216299 and bug 1216576

Comment 2

[Rail Aliiev [:rail]]

Attachment: release_promotion-tools.diff

the release-runner part

Comment 3

[Jordan Lund (:jlund)]

Comment on attachment 8676420 [details] [diff] [review]
release_promotion-tools.diff

Review of attachment 8676420 [details] [diff] [review]:
-----------------------------------------------------------------

<stamp>

::: buildfarm/release/release-runner.py
@@ +156,5 @@
>  
> +def get_en_US_config(release, branchConfig, branch, index):
> +    platforms = {}
> +    for platform in branchConfig["release_platforms"]:
> +        task = index.findTask("buildbot.revisions.{revision}.{branch}.{platform}".format(

I wonder if this should be a rig this out as a helper method now that get_l10n_config and get_en_US_config use have this same logic. I guess it depends if other things will need to find tasks in a similar fashion.

Comment 4

[Rail Aliiev [:rail]]

Comment on attachment 8676419 [details] [review]
do it!

Merged

Comment 5

[Rail Aliiev [:rail]]

Comment on attachment 8676420 [details] [diff] [review]
release_promotion-tools.diff

https://hg.mozilla.org/build/tools/rev/bca44606b1c5

Comment 6

[Rail Aliiev [:rail]]

Attachment: l10n partials

Treeherder view:
https://treeherder.allizom.org/#/jobs?repo=date&revision=1f53104e1641

release task graph:
https://tools.taskcluster.net/task-graph-inspector/#jIQQpzMGRoma0qp6KT2iBA/

Comment 7

[Justin Wood (:Callek)]

Comment on attachment 8702073 [details] [review]
l10n partials

commented in github