x/build: modify contents of "gray box" (summary markdown) in LUCI UI to be more directly to the point for test failures #67165
Comments
|
On https://go-review.googlesource.com/c/go/+/583455 PS1:
Can these URLs be changed to https://ci.chromium.org/ui/p/golang/builders/try/gotip-linux-386/b8748684851461439681/test-results instead? You showed me how to change the default behavior to not go to the Overview page, but we know the Test Results are the right landing page for users viewing a failure on a Gerrit page. Let's link directly there, so that we don't have to train everyone to change their defaults. (If everyone has to change their defaults, the defaults are wrong.) |
|
For the record, these are the magic places to click to get one-click viewing of test results. 
The first time you click on one of those, you have to also edit your LUCI config to default to "Test Results": 
Editing the config works even if you are not logged in. It stores the result in Local Storage. Mine says:
|
cherrymui
added
the
NeedsFix
Currently the LUCI UI for Go builds isn't very direct, especially in the common case of a test failure.
For example:
The "FAILURE" line is redundant, the "exit status 1" line is also not very useful (more useful for an infra failure, less useful for a test failure), and the "Links" section has only one link that goes to a more verbose set of information. Nowhere here do we link to the test results directly (we do for main Go repo builds, but that also includes less useful information, like the shard number).
@rsc proposed including a summary of the test failure text instead, which I believe is the right call. We already plumb around the fact that a test failed anyway.
I'll defend the existence of the
go on <x>andwebsite on <y>lines, because I don't believe that information is surfaced very well anywhere else in the UI, but other than that, the summary markdown could use a rethink.The text was updated successfully, but these errors were encountered: