dashboard: shard openbsd-amd64-68 TryBot also
The goal in CL 279512 was to reduce numTryTestHelpers from 5 to 4,
but it unintentionally set the field only in the 386 builder, not
the amd64 one. This caused the amd64 OpenBSD TryBot to get very
slow (20-25 minutes), making it very much a bottleneck for overall
TryBot completion time.
Also stop explicitly setting some fields in openbsd-amd64-62
builder that have no effect by now, to simplify configuration.
For golang/go#17104.
Updates golang/go#35712.
Change-Id: I22823e4848cab65f11bde2a1cc70527929d0792d
Reviewed-on: https://go-review.googlesource.com/c/build/+/279723
Trust: Dmitri Shuralyov <dmitshur@golang.org>
Run-TryBot: Dmitri Shuralyov <dmitshur@golang.org>
TryBot-Result: Go Bot <gobot@golang.org>
Reviewed-by: Alexander Rakoczy <alex@golang.org>
diff --git a/dashboard/builders.go b/dashboard/builders.go
index 04afa8d..5a0cffa 100644
--- a/dashboard/builders.go
+++ b/dashboard/builders.go
@@ -2019,14 +2019,11 @@
Name: "openbsd-amd64-62",
HostType: "host-openbsd-amd64-62",
distTestAdjust: noTestDirAndNoReboot,
- tryBot: nil,
buildsRepo: func(repo, branch, goBranch string) bool {
// This builder is unfortunately still used by Go 1.15 and 1.14,
// so keep it around a bit longer. See golang.org/issue/42426.
return atMostGo1(goBranch, 15) && buildRepoByDefault(repo)
},
- numTestHelpers: 0,
- numTryTestHelpers: 5,
})
addBuilder(BuildConfig{
Name: "openbsd-amd64-64",
@@ -2040,10 +2037,11 @@
distTestAdjust: noTestDirAndNoReboot,
})
addBuilder(BuildConfig{
- Name: "openbsd-amd64-68",
- HostType: "host-openbsd-amd64-68",
- tryBot: defaultTrySet(),
- distTestAdjust: noTestDirAndNoReboot,
+ Name: "openbsd-amd64-68",
+ HostType: "host-openbsd-amd64-68",
+ tryBot: defaultTrySet(),
+ distTestAdjust: noTestDirAndNoReboot,
+ numTryTestHelpers: 4,
})
addBuilder(BuildConfig{
Name: "openbsd-386-68",