Closed Bug 1180050 Opened 9 years ago Closed 9 years ago

OrangeFactor should link to the Treeherder log viewer from the per-bug details page

Categories

(Tree Management Graveyard :: OrangeFactor, defect, P2)

defect

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: emorley, Assigned: emorley)

References

Details

Attachments

(1 file, 1 obsolete file)

Currently the bug comments are the easiest way to get to the log - without them, people have to go via Treeherder, filter down to the correct job and open the log that way. We should link to the log in OrangeFactor - there's a "logfile" property already in the ES doc - but Treeherder is currently always populating it with "00000000". Once we make Treeherder give us the actual log URL, we can easily add it to the API response: https://hg.mozilla.org/automation/orangefactor/file/f0b9277fc277/server/handlers.py#l334 ...and then the UI: https://hg.mozilla.org/automation/orangefactor/file/f0b9277fc277/html/scripts/woo.bugs.js#l240 I guess the question now is just whether we link to the raw log, or treeherder's log viewer? The latter has a greater log retention time (albeit just for the error summary excerpt) and highlighting.
Depends on: 1180052
Have revised the approach here and in bug 1180052: Treeherder will now provide the job_id, so OrangeFactor can just construct the Treeherder log viewer URL for each failure occurrence. I've also moved the Treeherder single push URL generation to a helper function at the same time, to improve readability. This won't have anything to display until bug 1180052 lands.
Attachment #8629168 - Attachment is obsolete: true
Attachment #8629396 - Flags: review?(jgriffin)
Summary: OrangeFactor should link to the job log on the per-bug details page → OrangeFactor should link to the Treeherder log viewer from the per-bug details page
Attachment #8629396 - Flags: review?(jgriffin) → review+
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Product: Tree Management → Tree Management Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: