Skip to content

Commit

Permalink
dashboard/buildlet: set GOTESTONLY=std on plan9
Browse files Browse the repository at this point in the history
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 <[email protected]>
  • Loading branch information
bradfitz committed Jan 9, 2015
1 parent 75337c4 commit 727ecd5
Showing 1 changed file with 10 additions and 0 deletions.
10 changes: 10 additions & 0 deletions dashboard/buildlet/buildlet.go
Original file line number Diff line number Diff line change
Expand Up @@ -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 {
Expand Down

0 comments on commit 727ecd5

Please sign in to comment.