reuse beetmover transforms in beetmover-checksums kind

NEW
Unassigned

Status

2 years ago
4 months ago

People

(Reporter: mtabara, Unassigned)

Tracking

Firefox Tracking Flags

(Not tracked)

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

2 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

a year ago
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)
(Reporter)

Comment 4

a year 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)
OK, just triaging the component.  I can leave it open for a bit longer.
(Reporter)

Comment 6

a year ago
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)
(Reporter)

Comment 8

10 months 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

9 months ago
Product: TaskCluster → Firefox Build System
(Reporter)

Comment 9

4 months 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
You need to log in before you can comment on or make changes to this bug.