dashboard: re-enable js/wasm TryBot (with Node 18)
After all.bash is passing with Node.js 18, the js/wasm builder can be
enabled as a regular TryBot again (this time with Node 18 in place of
Node 14).
For golang/go#57614.
Change-Id: I714380fba89e37275aed4144a1e8b50e0cc57884
Reviewed-on: https://go-review.googlesource.com/c/build/+/464036
Auto-Submit: Dmitri Shuralyov <dmitshur@golang.org>
TryBot-Result: Gopher Robot <gobot@golang.org>
Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
Run-TryBot: Dmitri Shuralyov <dmitshur@golang.org>
Reviewed-by: Michael Knyszek <mknyszek@google.com>
Reviewed-by: Carlos Amedee <carlos@golang.org>
diff --git a/dashboard/builders.go b/dashboard/builders.go
index eaae358..c3e8936 100644
--- a/dashboard/builders.go
+++ b/dashboard/builders.go
@@ -1583,7 +1583,6 @@
addMiscCompile("-other-1", "dragonfly-amd64", "linux-loong64")
addMiscCompile("-other-2", "linux-riscv64", "linux-s390x", "linux-arm-arm5") // 'linux-arm-arm5' is linux/arm with GOARM=5.
addMiscCompileGo1(20, "-go1.20", "freebsd-riscv64")
- addMiscCompileGo1(21, "-go1.21", "js-wasm") // TODO(go.dev/issue/57614): Use a misc-compile trybot for js/wasm until Node 18 builder can work as a trybot.
// TODO: Issue 25963, get the misc-compile trybots for Android/iOS.
// Then consider subrepos too, so "mobile" can at least be included
@@ -1904,10 +1903,9 @@
},
})
addBuilder(BuildConfig{
- Name: "js-wasm-node18",
- HostType: "host-linux-amd64-js-wasm-node18",
- KnownIssues: []int{57614}, // After this known issue is resolved, re-enable it as a TryBot below:
- //tryBot: explicitTrySet("go"),
+ Name: "js-wasm-node18",
+ HostType: "host-linux-amd64-js-wasm-node18",
+ tryBot: explicitTrySet("go"),
buildsRepo: func(repo, branch, goBranch string) bool {
b := buildRepoByDefault(repo) && atLeastGo1(goBranch, 21)
switch repo {
diff --git a/dashboard/builders_test.go b/dashboard/builders_test.go
index 3f48c88..9d10cd5 100644
--- a/dashboard/builders_test.go
+++ b/dashboard/builders_test.go
@@ -94,6 +94,7 @@
branch: "master",
want: []string{
"freebsd-amd64-12_3",
+ "js-wasm-node18",
"linux-386",
"linux-amd64",
"linux-amd64-boringcrypto",
@@ -119,7 +120,6 @@
"misc-compile-other-1",
"misc-compile-other-2",
"misc-compile-go1.20",
- "misc-compile-go1.21",
},
},
{
@@ -127,6 +127,7 @@
branch: "release-branch.go1.21",
want: []string{
"freebsd-amd64-12_3",
+ "js-wasm-node18",
"linux-386",
"linux-amd64",
"linux-amd64-boringcrypto",
@@ -152,7 +153,6 @@
"misc-compile-other-1",
"misc-compile-other-2",
"misc-compile-go1.20",
- "misc-compile-go1.21",
// Include longtest builders on Go repo release branches. See issue 37827.
"linux-386-longtest",
@@ -501,8 +501,8 @@
{b("android-amd64-emu", "go"), onlyPost},
{b("android-386-emu", "go"), onlyPost},
- {b("js-wasm-node18", "go"), onlyPost},
- {b("js-wasm-node18@go1.21", "go"), onlyPost},
+ {b("js-wasm-node18", "go"), both},
+ {b("js-wasm-node18@go1.21", "go"), both},
{b("js-wasm-node18@go1.20", "go"), none},
// Keep using js-wasm builder (with Node 14) only for Go 1.20 and older:
{b("js-wasm", "go"), none},