dashboard: skip reboot test on ppc64x builders
Updates golang/go#35233
Change-Id: Icfa19da5d0dd3c47ffa94d11f069b8836c86cdcc
Reviewed-on: https://go-review.googlesource.com/c/build/+/204038
Reviewed-by: Bryan C. Mills <bcmills@google.com>
diff --git a/dashboard/builders.go b/dashboard/builders.go
index b0af586..e1a53f6 100644
--- a/dashboard/builders.go
+++ b/dashboard/builders.go
@@ -2208,19 +2208,22 @@
Notes: "Oracle Solaris release version",
})
addBuilder(BuildConfig{
- Name: "linux-ppc64-buildlet",
- HostType: "host-linux-ppc64-osu",
- FlakyNet: true,
+ Name: "linux-ppc64-buildlet",
+ HostType: "host-linux-ppc64-osu",
+ FlakyNet: true,
+ shouldRunDistTest: ppc64DistTestPolicy,
})
addBuilder(BuildConfig{
- Name: "linux-ppc64le-buildlet",
- HostType: "host-linux-ppc64le-osu",
- FlakyNet: true,
+ Name: "linux-ppc64le-buildlet",
+ HostType: "host-linux-ppc64le-osu",
+ FlakyNet: true,
+ shouldRunDistTest: ppc64DistTestPolicy,
})
addBuilder(BuildConfig{
- Name: "linux-ppc64le-power9osu",
- HostType: "host-linux-ppc64le-power9-osu",
- FlakyNet: true,
+ Name: "linux-ppc64le-power9osu",
+ HostType: "host-linux-ppc64le-power9-osu",
+ FlakyNet: true,
+ shouldRunDistTest: ppc64DistTestPolicy,
})
addBuilder(BuildConfig{
Name: "linux-arm64-packet",
@@ -2435,6 +2438,17 @@
return true
}
+// ppc64DistTestPolicy is a shouldRunDistTest policy function
+// that's shared by linux-ppc64le, -ppc64le-power9osu, and -ppc64.
+func ppc64DistTestPolicy(distTest string, isTry bool) bool {
+ if distTest == "reboot" {
+ // Skip test. It seems to use a lot of memory?
+ // See https://golang.org/issue/35233.
+ return false
+ }
+ return true
+}
+
// TryBuildersForProject returns the builders that should run as part of
// a TryBot set for the given project.
// The project argument is of the form "go", "net", "sys", etc.