unstub funsize mar urls in release promotion

RESOLVED FIXED

Status

defect
RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: bhearsum, Assigned: rail)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(3 attachments)

Funsize jobs are kicked off now, but they don't work due to bad URLs!
Assignee: nobody → rail
Posted file do it!
This also addresses bug 1216299 and bug 1216576
Attachment #8676419 - Flags: review?(jlund)
the release-runner part
Attachment #8676420 - Flags: review?(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.
Attachment #8676420 - Flags: review?(jlund) → review+
Attachment #8676419 - Flags: review?(jlund) → review+
Comment on attachment 8676419 [details] [review]
do it!

Merged
Attachment #8676419 - Flags: checked-in+
Depends on: 1216152
Comment on attachment 8702073 [details] [review]
l10n partials

commented in github
Attachment #8702073 - Flags: review?(bugspam.Callek) → review+
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.