Closed Bug 1213283 Opened 9 years ago Closed 9 years ago

Add display only "summary" option to |mach taskcluster-graph|

Categories

(Taskcluster :: General, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: ahal, Assigned: ahal)

Details

Attachments

(1 file)

It would be handy to have a command line option that only displays a human readable summary of the graph for debugging purposes. As it stands, you have to sift through a massive json structure or grep the magic builder names to see if the graph contains the proper jobs.
Bug 1213283 - Add option to only print task names in |mach taskcluster-graph|, r=dustin
Attachment #8671946 - Flags: review?(dustin)
Comment on attachment 8671946 [details]
MozReview Request: Bug 1213283 - Add option to only print task names in |mach taskcluster-graph|, r=dustin

https://reviewboard.mozilla.org/r/21657/#review19465

Too cool - thanks!

I wonder if there's a way to have the decision task also print this out, separately from outputting the graph..
Attachment #8671946 - Flags: review?(dustin) → review+
Yeah, that would be useful! We could add some kind of --output-file argument where if specified, the graph is saved there instead of stdout. Then the human readable version can go to stdout instead.

Sort of related.. I'm not entirely sure why 'taskcluster-graph' is a mach command in the first place, as opposed to a module that gets called directly by the decision (without going through mach). We could still keep the mach command around for local debugging, but wouldn't have to worry about breaking things in automation.

I'll land this patch for now and file a follow up to tackle the rest.
Yeah, there's been some discussion of rewriting the graph generation to be in JS (still in-tree, but not part of mach).

That would make it easier to do things like make TC API calls (so adding the graph directly, rather than spitting JSON to an artifact and relying on the worker) and also give a nice JSON-y language to use for templating.

It's still a rather vague idea.
https://hg.mozilla.org/mozilla-central/rev/bac359fbd7f5
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: