1125094 - Change the "log not parsed" message to include a link to the raw log

Status: VERIFIED FIXED

(Tree Management :: Treeherder, defect, P3)

Description

[Ed Morley [:emorley]]

If we haven't finished parsing the log - and so the Treeherder UI displays the "not parsed yet, will periodically refresh" message - we could help avoid user frustration by making it obvious how they can use the raw log in the meantime.

We should either:
a) Add "you can use the raw log button in the meantime" to the message (since I think people may not have seen the raw link button / think it's also deactivated like the log viewer one).
b) Or actually just link to the raw log in the explanation

Comment 1

[Jonathan French (:jfrench)]

How about:

Log status: No logs
Log status: Raw log available
Log status: Parsed

...with "Raw" a blue link to the raw log view
...with "Parsed" a blue link to the logviewer view

So both of those links act as an extra access point, in addition to our navbar icons.

Comment 2

[Jonathan French (:jfrench)]

Noting here also, with recent bumps in concurrency of log parser workers (2>5) and perhaps even more soon, I haven't seen a lot of log parsing 'in progress' states lately across m-c or m-i.

Not that we don't want the best possible UX if we do hit it.

Comment 3

[Ed Morley [:emorley]]

I was meaning more the message text on the right/center panel, rather than the log parsing status field in the job details panel on the left.

Comment 4

[Jonathan French (:jfrench)]

Ah, ok. I will have a look.

Comment 5

[Jonathan French (:jfrench)]

Attachment: logParsingInProgMsgProposed_rev1

Here's a proposed update, for feedback. Let me know if you'd like any tweaks to it :)

Comment 6

[Jonathan French (:jfrench)]

Comment on attachment 8602983 [details]
logParsingInProgMsgProposed_rev1

Ed's good with the change, so I'll open a PR.

Comment 7

[Jonathan French (:jfrench)]

Attachment: treeherder-ui-PR#516

Please see above PR for status and review.

Comment 8

[Ed Morley [:emorley]]

Comment on attachment 8603351 [details] [review]
treeherder-ui-PR#516

Looks good, thank you :-)

Comment 9

[Treeherder GitHub Bugbot]

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

https://github.com/mozilla/treeherder-ui/commit/8c9c99d1c596b151e9487c0baaebc053a01c91e3
Bug 1125094 - Improve the log parsing in-progress message

Comment 10

[Jonathan French (:jfrench)]

Marking resolved fixed. I will verify on the next push to stage/prod.

Comment 11

[Jonathan French (:jfrench)]

The log parsing is so quick on stage/prod it's been difficult to verify a job in this transition state.

I was just running a local vagrant and happened to see this messaging doing exactly what it should with my own slower server, so marking Verified based on that.

Comment 12

[Treeherder GitHub Bugbot]

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

https://github.com/mozilla/treeherder/commit/f7d9430787b48f7c438df5318fd93128887dc10e
Bug 1125094 - Improve the log parsing in-progress message