Closed Bug 1506633 Opened 6 years ago Closed 6 years ago

"Job" link in log viewer is confusing

Categories

(Tree Management :: Treeherder, defect, P1)

defect

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: aryx, Assigned: camd)

References

Details

(Keywords: regression)

Attachments

(1 file)

So counter-intuitively, the "revision" link is currently the link that (once bug 1506630 is fixed at least) is meant to point to the specific job. The "job" row is the same job row that appears in the job info panel on the main jobs view (the same React component is being used in both places to reduce code duplication). The entry in the job detail panel is meant to filter by last 10 jobs, so it is currently working as "expected". I do agree that this is confusing and would be something we should perhaps tweak though.
Blocks: 1450045
Flags: needinfo?(cdawson)
Priority: -- → P1
Summary: "Job" link of log viewer supposed to link to failed job but shows most recent jobs of that kind → "Job" link in log viewer is confusing
Perhaps the link from the log viewer to the jobs view should be a button on the log viewer toolbar, outside of the job info metadata?
(In reply to Ed Morley [:emorley] from comment #2) > Perhaps the link from the log viewer to the jobs view should be a button on > the log viewer toolbar, outside of the job info metadata? Yeah, this is a good idea. I'll do this. Perhaps leave the revision link as well, since people may be used to that. But remove the link for filtering by the job signature/strings. Just make it text.
Assignee: nobody → cdawson
Status: NEW → ASSIGNED
Flags: needinfo?(cdawson)
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Component: Treeherder: Log Viewer → TreeHerder
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: