[build] cmd/watchflakes: update whole-build text log to match recent LUCI

0 views

Skip to first unread message

Cherry Mui (Gerrit)

unread,

Feb 24, 2026, 1:36:38 PM (14 hours ago) Feb 24

to goph...@pubsubhelper.golang.org, golang-...@googlegroups.com, Michael Knyszek, Go LUCI, Dmitri Shuralyov, golang-co...@googlegroups.com

Cherry Mui submitted the change

Change information

Commit message:

cmd/watchflakes: update whole-build text log to match recent LUCI

For the whole-build text log, i.e. the "log" field in the query
language, we used to fetch a link from LUCI that is named with
"(combined output)". Something changed on the LUCI side, and the
link is now named like "go tool dist test -json output [shard 1]"
or "test golang.org/x/tools/gopls module output". Apparently, now
all the links from a build are relevant to some failures. So,
instead of name filtering, we just fetch all the logs and
concatenate them (in most builds there is only one). This log is
helpful when there are failures that are not attributed to
individual tests, e.g. some panics or timeouts. For example, it
will be able to read the panic in
https://ci.chromium.org/b/8690333242133168193 and match it to
"log ~ `SIGILL`".

Change-Id: Ic2cf580da43a918572e99343679805ecb176a633

Reviewed-on: https://go-review.googlesource.com/c/build/+/747541

LUCI-TryBot-Result: Go LUCI <golang...@luci-project-accounts.iam.gserviceaccount.com>

Reviewed-by: Michael Knyszek <mkny...@google.com>

Files: