Move taskcluster mach commands into a standalone module

RESOLVED WONTFIX

Status

Taskcluster
General
RESOLVED WONTFIX
3 years ago
2 years ago

People

(Reporter: ahal, Unassigned)

Tracking

Details

(Reporter)

Description

3 years ago
As far as I can tell, the taskcluster mach commands are mostly used by automation. E.g the decision task calls |mach taskcluster-graph| here:
https://dxr.mozilla.org/mozilla-central/source/testing/taskcluster/tasks/decision/branch.yml#70

But there's no particular reason that it should be going through mach. A good pattern is to have all functional logic living in a standalone python module that both the automation and mach can call into.

Though in this case, it's not obvious to me that these commands are useful to anyone but a small handful of people, so I'm not sure it's worth cluttering up |mach help|.
I don't think we have a bug for it yet, because it hasn't become a priority, but we're looking at a way to generate task graphs more maintainably -- the plethora of copy/pasted yml files with inheritance is not making anyone happy.

We've talked about using https://github.com/jonasfj/json-parameterization and generating the graph using a JS app, rather than Python.  That would still be in-tree, but would facilitate both easy handling of JSON and outgoing calls to taskcluster APIs (e.g., createTask, createTaskGraph).

Maybe this bug gets transmuted into that one?
Andrew, based on our discsussions around bu 1258497, has this improved?  I'm still using `mach taskgraph decision`, but the alternative is to invoke something like `PYTHONPATH=here:there:another-place /path/to/python2.7 taskcluster/taskgraph.py` which just seems a lot messier.
Flags: needinfo?(ahalberstadt)
(Reporter)

Comment 3

2 years ago
Yeah, you mentioned that those commands are still useful to developers, and you moved almost all the logic to an external module. So I think this can be WONTFIX'ed.
Status: NEW → RESOLVED
Last Resolved: 2 years ago
Flags: needinfo?(ahalberstadt)
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.