From 04319633a9af49b3c2a629fbbbf15bd0b0386d14 Mon Sep 17 00:00:00 2001 From: Brad Fitzpatrick Date: Wed, 7 Jan 2015 20:56:15 -0800 Subject: [PATCH] dashboard/buildlet: set GOTESTONLY=std on plan9 This resulted in our first "ok" on the dashboard for Plan 9 with the buildlet, in 19 minutes. It only runs the std tests, and nothing else after that. Update golang/go#9491 Change-Id: Iad77a594f83bfd3fa72596bcc3057645d9c9bb4c Reviewed-on: https://go-review.googlesource.com/2523 Reviewed-by: Andrew Gerrand --- buildlet/buildlet.go | 10 ++++++++++ 1 file changed, 10 insertions(+) diff --git a/buildlet/buildlet.go b/buildlet/buildlet.go index 4a9047d43c..e96f236aa0 100644 --- a/buildlet/buildlet.go +++ b/buildlet/buildlet.go @@ -66,6 +66,16 @@ func main() { if !metadata.OnGCE() && !strings.HasPrefix(*listenAddr, "localhost:") { log.Printf("** WARNING *** This server is unsafe and offers no security. Be careful.") } + if runtime.GOOS == "plan9" { + // Plan 9 is too slow on GCE, so stop running run.rc after the basics. + // See https://golang.org/cl/2522 and https://golang.org/issue/9491 + // TODO(bradfitz): once the buildlet has environment variable support, + // the coordinator can send this in, and this variable can be part of + // the build configuration struct instead of hard-coded here. + // But no need for environment variables quite yet. + os.Setenv("GOTESTONLY", "std") + } + if *scratchDir == "" { dir, err := ioutil.TempDir("", "buildlet-scatch") if err != nil {