Closed
Bug 1166984
Opened 9 years ago
Closed 9 years ago
I think Taskcluster jobs' "Inspect Task" link should still be in the left panel, not the Job Details tab
Categories
(Tree Management :: Treeherder, defect, P3)
Tree Management
Treeherder
Tracking
(Not tracked)
VERIFIED
FIXED
People
(Reporter: KWierso, Assigned: jfrench)
References
Details
Attachments
(1 file)
I think it should stay in the left pane, closer (and without needing any unnecessary extra clicks) to the buttons for the logviewer links.
Assignee | ||
Comment 1•9 years ago
|
||
If other Sheriffs are in agreement, should we also the default height of the main job panel slightly taller(?), so this added Inspect Task entry isn't clipped off the bottom of the bottom left pane (and then requires equivalent effort to view it).
I believe in some cases this change will also result in an empty Job Details tab, where afaik currently we always had something populating it. So we may need to add a "Job Details empty" tab message or similar.
Flags: needinfo?(ryanvm)
Flags: needinfo?(cbook)
Comment 2•9 years ago
|
||
I think we need to be careful about just polling the sheriffs on this-- we added the job details panel because the stuff on the left side wasn't discoverable enough to other users of treeherder.
I'm fine with just including the information in two different places fwiw. I don't think that hurts.
Assignee | ||
Comment 3•9 years ago
|
||
Couple of options perhaps:
a) conditionally provide a 'task' icon in the navbar eg. beside the current Log icon, if the job is a taskcluster job? The navbar UI might bounce around a lot though when traversing buildbot and tc jobs.
b) perhaps a more minimal "task" text link in the bottom left details, similar and adjacent to our 'sig'-nature link. So we don't consume another full line.
Assignee | ||
Updated•9 years ago
|
Flags: needinfo?(ryanvm)
Assignee | ||
Updated•9 years ago
|
OS: Unspecified → All
Priority: -- → P3
Hardware: Unspecified → All
Updated•9 years ago
|
Flags: needinfo?(cbook)
Assignee | ||
Updated•9 years ago
|
Assignee: nobody → tojonmz
Status: NEW → ASSIGNED
Assignee | ||
Comment 4•9 years ago
|
||
I think we'll go with a regular line entry below the Log parsing status for now in Job details panel. Based on discussions with Wes, sheriffs run at sufficient screen res systems the default 35% height for info panel will accommodate the extra line.
Assignee | ||
Comment 5•9 years ago
|
||
Please see above PR for status/review.
Attachment #8633475 -
Flags: review?(wlachance)
Comment 6•9 years ago
|
||
Comment on attachment 8633475 [details]
PR 755
lgtm
Attachment #8633475 -
Flags: review?(wlachance) → review+
Comment 7•9 years ago
|
||
Commit pushed to master at https://github.com/mozilla/treeherder
https://github.com/mozilla/treeherder/commit/9230572e17bfdaa169f7234adaa8859411c6947f
Bug 1166984 - Provide inspect-task link in Job details panel
Assignee | ||
Comment 8•9 years ago
|
||
Marking fixed per above merge. Will verify on the next push to stage/prod.
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.
Description
•