x/build/cmd/coordinator: invalid SlowBots in TRY=
comments are silently ignored
#58902
Labels
Milestone
TRY=
comments are silently ignored
#58902
In https://go-review.git.corp.google.com/c/go/+/465805/6#message-b69a437c43e08f518fc3ffb4de60cc525f9ee9ef, I typo'd a
TRY=
line, listing a valid builder name (longtest
) along with an invalid one (windows-awd64-longtest
).The Gopher Robot ran only the
longtest
builder, and happily added aTryBot-Result
along with a list of slowbots that included only the valid one.Especially given that the
TryBot+1
now feeds into submitting auto-submit CLs, it should not ignore invalid builders. Instead, it should reply on the CL with an explicit error if theTRY=
line contains an invalid name.This is related to #41831, but perhaps easier to resolve.
(attn @golang/release)
The text was updated successfully, but these errors were encountered: