Skip to content

Commit c640e78

Browse files
committed
cmd/watchflakes: ignore builders with known issue
Builders with a known issue are in development and are not fully functioning. They may encounter a lot of failures. Skip them. Change-Id: Ia89c86975165a05a1f2a0a382d09c86c0ba6d356 Reviewed-on: https://go-review.googlesource.com/c/build/+/602576 LUCI-TryBot-Result: Go LUCI <[email protected]> Reviewed-by: Dmitri Shuralyov <[email protected]> Reviewed-by: Dmitri Shuralyov <[email protected]>
1 parent a4d4fa3 commit c640e78

File tree

1 file changed

+7
-1
lines changed

1 file changed

+7
-1
lines changed

cmd/watchflakes/luci.go

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -194,7 +194,13 @@ nextPage:
194194
}
195195
for _, b := range resp.GetBuilders() {
196196
var p BuilderConfigProperties
197-
json.Unmarshal([]byte(b.GetConfig().GetProperties()), &p)
197+
err := json.Unmarshal([]byte(b.GetConfig().GetProperties()), &p)
198+
if err != nil {
199+
return nil, err
200+
}
201+
if p.KnownIssue != 0 {
202+
continue
203+
}
198204
if all || (p.Repo == repo && p.GoBranch == goBranch) {
199205
builders = append(builders, Builder{b.GetId().GetBuilder(), &p})
200206
}

0 commit comments

Comments
 (0)