Skip to content

Commit d14234e

Browse files
author
Bryan C. Mills
committed
dashboard: omit the website repo on plan9
The plan9 builders have relatively slow filesystems, and the website repo is relatively filesystem-intensive — many of its tests walk and process large trees of website content. Instead of trying to precisely tune the x/website tests to pass on the plan9 builders, let's skip this repo on them. For golang/go#49338. Change-Id: Ieaf50fb6955a6b39012a23ff72e61c1c31e4ce7c Reviewed-on: https://go-review.googlesource.com/c/build/+/362975 Trust: Bryan C. Mills <[email protected]> Run-TryBot: Bryan C. Mills <[email protected]> TryBot-Result: Go Bot <[email protected]> Reviewed-by: Carlos Amedee <[email protected]> Reviewed-by: Dmitri Shuralyov <[email protected]>
1 parent f077b9b commit d14234e

File tree

1 file changed

+17
-4
lines changed

1 file changed

+17
-4
lines changed

dashboard/builders.go

Lines changed: 17 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -2010,7 +2010,7 @@ func init() {
20102010
}
20112011
return run
20122012
},
2013-
buildsRepo: onlyMasterDefault,
2013+
buildsRepo: plan9Default,
20142014
})
20152015
addBuilder(BuildConfig{
20162016
Name: "windows-amd64-2008",
@@ -2489,7 +2489,7 @@ func init() {
24892489
Name: "plan9-arm",
24902490
HostType: "host-plan9-arm-0intro",
24912491
distTestAdjust: noTestDirAndNoReboot,
2492-
buildsRepo: onlyMasterDefault,
2492+
buildsRepo: plan9Default,
24932493
})
24942494
addBuilder(BuildConfig{
24952495
Name: "plan9-amd64-0intro",
@@ -2503,7 +2503,7 @@ func init() {
25032503
}
25042504
return run
25052505
},
2506-
buildsRepo: onlyMasterDefault,
2506+
buildsRepo: plan9Default,
25072507
})
25082508
addBuilder(BuildConfig{
25092509
Name: "plan9-386-0intro",
@@ -2517,7 +2517,7 @@ func init() {
25172517
}
25182518
return run
25192519
},
2520-
buildsRepo: onlyMasterDefault,
2520+
buildsRepo: plan9Default,
25212521
})
25222522
addBuilder(BuildConfig{
25232523
Name: "aix-ppc64",
@@ -2742,6 +2742,19 @@ func onlyMasterDefault(repo, branch, goBranch string) bool {
27422742
return branch == "master" && goBranch == "master" && buildRepoByDefault(repo)
27432743
}
27442744

2745+
// plan9Default is like onlyMasterDefault, but also omits repos that are
2746+
// both filesystem-intensive and unlikely to be relevant to plan9 users.
2747+
func plan9Default(repo, branch, goBranch string) bool {
2748+
switch repo {
2749+
case "website":
2750+
// The x/website tests read and check the website code snippets,
2751+
// which require many filesystem walk and read operations.
2752+
return false
2753+
default:
2754+
return onlyMasterDefault(repo, branch, goBranch)
2755+
}
2756+
}
2757+
27452758
// disabledBuilder is a buildsRepo policy function that always return false.
27462759
func disabledBuilder(repo, branch, goBranch string) bool { return false }
27472760

0 commit comments

Comments
 (0)