1087578 - Add link to reftest analyzer on logviewer page

Status: VERIFIED FIXED

(Tree Management :: Treeherder, defect, P4)

Description

[Kartikaya Gupta (email:[e-mail])]

Often for failures one of the sheriffs will post in IRC or on the bug a link like https://treeherder.mozilla.org/ui/logviewer.html#?job_id=3224196&repo=mozilla-inbound - this is nice because it provides the details of the failure right away, and also has a link back to the push that it came from. However it would be even better if there were a link to the reftest analyzer from this page (at least for reftest failures) so that it's easy to get to, without having to go back to changeset page and having to hunt for the right failure.

Comment 1

[Ed Morley [:emorley]]

Yeah agree this would be nice.

In general we should aim to sync up the job details panel features with the log viewer header - see also bug 1060482.

Comment 2

[Jonathan French (:jfrench)]

This was discussed on IRC again today. If I can get something working locally I'll assign myself :)

Comment 3

[Jonathan French (:jfrench)]

Attachment: treeherder-PR664

Please see above PR for status and review.

Comment 4

[Cameron Dawson [:camd]]

just a couple comments of things to try.  Clearing the review for now.  Thanks!!  :)

Comment 5

[Jonathan French (:jfrench)]

Comment on attachment 8624881 [details] [review]
treeherder-PR664

I made the changes discussed, I think we're ready for another look.

Comment 6

[Treeherder GitHub Bugbot]

Commit pushed to master at https://github.com/mozilla/treeherder

https://github.com/mozilla/treeherder/commit/d7474956b5aa41e0d937efeda7ddc60c229d2fc6
Bug 1087578 - Add a reftest link in logviewer

This adds a reftest analyzer button to logviewer, a provider for it
and for its equivalent reftest button in the job details panel.

Comment 7

[Jonathan French (:jfrench)]

Marking fixed per above merge. I will verify on the next push to stage/prod.

Comment 8

[Jonathan French (:jfrench)]

Verified fixed on stage. Still to be pushed to prod.