Open
Bug 1342254
Opened 7 years ago
Updated 1 year ago
reuse beetmover transforms in beetmover-checksums kind
Categories
(Firefox Build System :: Task Configuration, task)
Firefox Build System
Task Configuration
Tracking
(Not tracked)
NEW
People
(Reporter: mtabara, Unassigned)
Details
While doing other beetmover-scope related-work, I realized I could have done a better job of reusing some transforms in beetmover-checksums task[1]. More specifically, the main bulk of the task (task decription) can be easily reused from beetmover transforms [2]. I could reused that and instead have a tinier beetmover-checksums transforms, right at the start to set the Treeherder, locale, synbols, etc, similarly to how beetmover-l10n was done. [1]: https://hg.mozilla.org/mozilla-central/file/tip/taskcluster/ci/beetmover-checksums/kind.yml [2]: https://hg.mozilla.org/mozilla-central/file/tip/taskcluster/taskgraph/transforms/beetmover.py
Reporter | ||
Comment 1•7 years ago
|
||
By doing so, I could also get rid of one additional call of `get_beetmover_bucket_scope`from https://hg.mozilla.org/mozilla-central/rev/e1ea30263e2f.
Updated•7 years ago
|
Assignee: mtabara → nobody
Status: ASSIGNED → NEW
Comment 2•7 years ago
|
||
Rail, is this still worth doing? Something you or someone else can mentor?
Flags: needinfo?(rail)
Comment 3•7 years ago
|
||
Let's ask the person who filed this. :) 302 mtabara
Flags: needinfo?(rail) → needinfo?(mtabara)
Reporter | ||
Comment 4•7 years ago
|
||
I'd be happy to do it at some point this quarter, but my focus is elsewhere currently. It's not urgent so if this stands in the way, we can definitely close it.
Flags: needinfo?(mtabara)
Comment 5•7 years ago
|
||
OK, just triaging the component. I can leave it open for a bit longer.
Reporter | ||
Comment 6•7 years ago
|
||
Thanks! Moving this to my queue. For posterity, I plan to work on this during Q4 2017.
Assignee: nobody → mtabara
Status: NEW → ASSIGNED
Comment 7•6 years ago
|
||
Found in triage today. Mihai: is this still on your radar?
Flags: needinfo?(mtabara)
Reporter | ||
Comment 8•6 years ago
|
||
(In reply to Chris Cooper [:coop] from comment #7) > Found in triage today. > > Mihai: is this still on your radar? Yeah. Likely to handle it in the next two-three weeks when we'll tackle cleanup after BBB-downfall. It's part of our list[1]. Am happy to move it under RelEng::Release Automation if that is more suited. Thanks for the heads-up [1]: https://trello.com/c/fwJruFlS/186-rewrite-the-beetmover-dependency-system-its-dirty-and-not-flexible-enough-to-meet-our-needs
Flags: needinfo?(mtabara)
Updated•6 years ago
|
Product: TaskCluster → Firefox Build System
Reporter | ||
Comment 9•6 years ago
|
||
Not actively working on this. However, I think :Callek is making some great progress in refactoring the beetmover transforms so there is value in keeping this open for a little longer.
Assignee: mtabara → nobody
Status: ASSIGNED → NEW
Updated•2 years ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•