cmd/coordinator: fix trybot remaining builds count for subrepos
Fixes assumption broken by https://golang.org/cl/143538 where number
of builders (types of builders) no longer equals the number of trybot
builds. (We now run linux-amd64 1 or 3 times depending on the repo)
Fixes golang/go#28714
Change-Id: I3b85adbb79508890d16311fc75f4b48ffc1f3c78
Reviewed-on: https://go-review.googlesource.com/c/149437
Reviewed-by: Dmitri Shuralyov <dmitshur@golang.org>
diff --git a/cmd/coordinator/coordinator.go b/cmd/coordinator/coordinator.go
index 6129742..65aa6ce 100644
--- a/cmd/coordinator/coordinator.go
+++ b/cmd/coordinator/coordinator.go
@@ -1085,7 +1085,6 @@
tryKey: key,
tryID: "T" + randHex(9),
trySetState: trySetState{
- remain: len(builders),
builds: make([]*buildStatus, 0, len(builders)),
},
}
@@ -1108,6 +1107,7 @@
idx := len(ts.builds)
ts.builds = append(ts.builds, bs)
+ ts.remain++
go bs.start() // acquires statusMu itself, so in a goroutine
go ts.awaitTryBuild(idx, bs, brev)
}
