Skip to content

Commit 8713701

Browse files
committed
dashboard: remove alpine builder
It's doing no good being red all the time, especially as nobody is working on it. Updates golang/go#22689 Updates golang/go#19938 Change-Id: Icb947878d85e920f24ea458eb0f319844ca5bd60 Reviewed-on: https://go-review.googlesource.com/78575 Reviewed-by: Brad Fitzpatrick <[email protected]>
1 parent 2662697 commit 8713701

File tree

1 file changed

+9
-4
lines changed

1 file changed

+9
-4
lines changed

dashboard/builders.go

Lines changed: 9 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -895,10 +895,15 @@ func init() {
895895
numTestHelpers: 6, // As of 2017/05/16, 3 helpers are needed for tests and 3 more for benchmarks to complete in 5m.
896896
RunBench: true,
897897
})
898-
addBuilder(BuildConfig{
899-
Name: "linux-amd64-alpine",
900-
HostType: "host-linux-x86-alpine",
901-
})
898+
899+
const testAlpine = false // Issue 22689 (hide all red builders), Issue 19938 (get Alpine passing)
900+
if testAlpine {
901+
addBuilder(BuildConfig{
902+
Name: "linux-amd64-alpine",
903+
HostType: "host-linux-x86-alpine",
904+
})
905+
}
906+
902907
// Add the -vetall builder. The builder name suffix "-vetall" is recognized by cmd/dist/test.go
903908
// to only run the "go vet std cmd" test and no others.
904909
addBuilder(BuildConfig{

0 commit comments

Comments
 (0)