Improve error messages when the log is not available

RESOLVED FIXED

Status

P1
normal
RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: emorley, Assigned: camd)

Tracking

Details

Attachments

(2 attachments)

(Reporter)

Description

4 years ago
Created attachment 8490624 [details]
screenshot

For the red job here:
http://treeherder-dev.allizom.org/ui/#/jobs?repo=mozilla-central&revision=8252eae8278c&searchQuery=b2g_mozilla-central_emulator_dep

The job details panel does not link to the log viewer, nor the raw log. In addition, the failure summary tab says "Failure summary is empty". (See screenshot).

In TBPL we get "Log not available." in the failure summary, the brief log says "Log not available at URL <foo>", and the raw log link 404s.

If the log is non-existent, we should:
1) Say "Log not available" (or not found) rather than "Failure summary is empty" in the failure tab.
2) Still show the log viewer buttons, but perhaps disabled - like we do for parsing status "pending".
3) Show a parsing status of "failed" or similar (the parsing status line doesn't appear anywhere).
(Reporter)

Updated

4 years ago
Priority: P2 → P1
(Assignee)

Comment 1

4 years ago
Created attachment 8493406 [details]
with fixes
(Assignee)

Comment 3

4 years ago
both buttons have the "you can't do this" mouse pointer.

fixed in:
https://github.com/mozilla/treeherder-ui/commit/20c5cfd2f4d22620fe2bcc938e068738e70e9e6a
(Assignee)

Updated

4 years ago
Status: NEW → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
(Reporter)

Updated

4 years ago
Assignee: nobody → cdawson
(Reporter)

Comment 4

4 years ago
Thank you for this and the other related "Summary is empty" bugs - minor change but big difference to UX :-)
You need to log in before you can comment on or make changes to this bug.