Closed Bug 1032504 Opened 10 years ago Closed 10 years ago

Link to the raw log & treeherder UI from the log viewer

Categories

(Tree Management :: Treeherder, defect, P2)

defect

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: emorley, Assigned: jfrench, Mentored)

References

()

Details

(Whiteboard: [good_first_bug][mentor=camd][lang=html])

It would be really handy if the log viewer, eg: https://treeherder.mozilla.org/ui/logviewer.html#?job_id=320916&repo=mozilla-inbound

...linked to:
* The raw log (eg http://ftp.mozilla.org/pub/mozilla.org/mobile/tinderbox-builds/mozilla-inbound-android/1404161907/mozilla-inbound_ubuntu64_vm_mobile_test-mochitest-10-bm115-tests1-linux64-build5.txt.gz)
* It's resultset on treeherder (eg https://treeherder.mozilla.org/ui/#/jobs?repo=mozilla-inbound&revision=6da4e0b92328)

Since it's a pretty frequent workflow for the sheriffs to have to switch between those views (eg someone pastes a link in IRC to a failed job's parsed log and we have to switch to other views to further analyse).

As part of this, I think we could maybe tweak the list of attributes displayed on the left panel, eg:

builder	mozilla-inbound_ubuntu64_vm_mobile_test-mochitest-10
buildid	20140630135827
builduid	6ada630172f94649bbac00af3acab625
results	warnings (1)
revision	6da4e0b92328
slave	tst-linux64-spot-522
starttime	1404163741.02

And maybe add/remove a few things and make some link to the items mentioned above. (But I'll have to have more of a think).
Priority: -- → P2
Mentor: cdawson
Whiteboard: [good first bug][mentor=camd][lang=html]
Whiteboard: [good first bug][mentor=camd][lang=html] → [good_first_bug][mentor=camd][lang=html]
I'll have a look at this, and see what I can do.
Assignee: nobody → tojonmz
Status: NEW → ASSIGNED
Item #2 - its revision link - has been landed on master and will be pushed later today
https://github.com/mozilla/treeherder-ui/pull/96
merged to master
Item #1 - the raw log - has been landed on master and will also be pushed shortly
https://github.com/mozilla/treeherder-ui/pull/99

Once both are up I will re-test on production and mark this bug fixed. If there's any other items to add to the left panel attributes per Ed's thoughts in Comment#1, perhaps we can open that as a separate bug.
Both PR's looks correct on all servers (dev/stage/prod) so marking this bug Fixed.
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.