Skip to content

Commit cd2ea43

Browse files
committed
dashboard: speed up js/wasm trybots
We don't need to run all the tests in a trybot. Run just enough to get good coverage, without going over 5 minutes. Any regressions elsewhere will be caught by the build.golang.org (slower) runs. Change-Id: I32f1fc17681bfb509844d2bd35b05c950806d283 Reviewed-on: https://go-review.googlesource.com/121938 Reviewed-by: Brad Fitzpatrick <[email protected]>
1 parent 114c42e commit cd2ea43

File tree

1 file changed

+20
-3
lines changed

1 file changed

+20
-3
lines changed

dashboard/builders.go

Lines changed: 20 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1155,9 +1155,26 @@ func init() {
11551155
env: []string{"GOOS=nacl", "GOARCH=amd64p32", "GOHOSTOS=linux", "GOHOSTARCH=amd64"},
11561156
})
11571157
addBuilder(BuildConfig{
1158-
Name: "js-wasm",
1159-
HostType: "host-js-wasm",
1160-
TryBot: true,
1158+
Name: "js-wasm",
1159+
HostType: "host-js-wasm",
1160+
TryBot: true,
1161+
ShouldRunDistTest: func(distTest string, isTry bool) bool {
1162+
if isTry {
1163+
if strings.HasPrefix(distTest, "test:") {
1164+
return false
1165+
}
1166+
if strings.Contains(distTest, "/internal/") ||
1167+
strings.Contains(distTest, "vendor/golang.org/x/arch") {
1168+
return false
1169+
}
1170+
switch distTest {
1171+
case "cmd/go", "nolibgcc:crypto/x509":
1172+
return false
1173+
}
1174+
return true
1175+
}
1176+
return true
1177+
},
11611178
numTryTestHelpers: 4,
11621179
GoDeps: []string{
11631180
"3dced519cbabc213df369d9112206986e62687fa", // first passing commit

0 commit comments

Comments
 (0)