Closed Bug 1442684 Opened 2 years ago Closed 8 months ago

refactor beetmoverscript to strip it of all the hacks

Categories

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

enhancement
Not set

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: mtabara, Assigned: mtabara)

References

(Depends on 1 open bug)

Details

Attachments

(1 file)

In bug 1424482 :jlorenzo added support for in-tree payload which makes it easy to be consistent with several tasks as the former way of doing it, via balrog_props, basically sucks and is unreliable the more we need more products (e.g TB, more fennec, etc).

For that reason, we need to:
* prep a PR that kills balrog_props altogether
* all in-tree tasks need to have relprops payload, correctly specified
* all the hacks/checks depending on platform/stage_platform need to go away as soon as possible

More motivation for this lies within TB difficulties to set those fields properly. See bug 1437586 for more details.

Other changes that could be implemented along the way are:
* https://trello.com/c/fwJruFlS/186-rewrite-the-beetmover-dependency-system-its-dirty-and-not-flexible-enough-to-meet-our-needs
* https://trello.com/c/Soc8Pz6X/243-beetmoverscript-other-improvements
Assignee: nobody → mtabara
Depends on: 1443588
Bug 1445910 will remove balrog_props from tree. Let's make sure we cleanup beetmover by then too, targeting 61.
See Also: → 1445910
Note to self: to reiterate, reread https://trello.com/c/Soc8Pz6X/243-1442684-beetmoverscript-other-improvements for more ideas on what to get along there.
Component: Release Automation: Other → Release Automation: Uploading
Some of this are to be addressed with upcoming work for declarative artifacts.
Depends on: 1466714
Comment on attachment 9020733 [details] [review]
[beetmoverscript] strip of bouncer.apk  as it seems unused

r+ by Callek and Johan in the PR.
Landed to production with the 7.10.1 bump today.
Attachment #9020733 - Flags: review+
Attachment #9020733 - Flags: checked-in+
Most of bugs are now done here. We can close this.
Status: NEW → RESOLVED
Closed: 8 months ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.