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
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.
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.
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.
Ah, ok. I will have a look.
Created attachment 8602983 [details] logParsingInProgMsgProposed_rev1 Here's a proposed update, for feedback. Let me know if you'd like any tweaks to it :)
Comment on attachment 8602983 [details] logParsingInProgMsgProposed_rev1 Ed's good with the change, so I'll open a PR.
Created attachment 8603351 [details] [review] treeherder-ui-PR#516 Please see above PR for status and review.
Comment on attachment 8603351 [details] [review] treeherder-ui-PR#516 Looks good, thank you :-)
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
Marking resolved fixed. I will verify on the next push to stage/prod.
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.
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