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)

task

Tracking

(Not tracked)

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
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.
Assignee: mtabara → nobody
Status: ASSIGNED → NEW
Rail, is this still worth doing?  Something you or someone else can mentor?
Flags: needinfo?(rail)
Let's ask the person who filed this. :)

302 mtabara
Flags: needinfo?(rail) → needinfo?(mtabara)
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)
OK, just triaging the component.  I can leave it open for a bit longer.
Thanks! Moving this to my queue. For posterity, I plan to work on this during Q4 2017.
Assignee: nobody → mtabara
Status: NEW → ASSIGNED
Found in triage today.

Mihai: is this still on your radar?
Flags: needinfo?(mtabara)
(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)
Product: TaskCluster → Firefox Build System
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
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.