Skip to content

Commit

Permalink
build: restore original timeouts in run.rc
Browse files Browse the repository at this point in the history
The timeouts were increased in CL 2462 and CL 2510
to work around a slowness issue when running Go
programs on a Plan 9 machine on GCE.

Since we figured out this issue, we can restore
the timeouts to their original values.

Updates golang#10028.

Change-Id: I2e5b91666461715df69df97ea791f3d88d9de4d0
Reviewed-on: https://go-review.googlesource.com/6261
Reviewed-by: Brad Fitzpatrick <[email protected]>
  • Loading branch information
0intro committed Feb 27, 2015
1 parent d68f52a commit c62b003
Showing 1 changed file with 2 additions and 6 deletions.
8 changes: 2 additions & 6 deletions src/run.rc
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@ if not {
GOROOT_FINAL = ()
echo '# Testing packages.'
time go test std cmd -short -timeout 240s
time go test std cmd -short -timeout 120s
echo
# Temporary GCE builder hack until Plan 9 on GCE is fast enough.
Expand All @@ -42,12 +42,8 @@ if(~ $GOTESTONLY std) {
# We set GOMAXPROCS=2 in addition to -cpu=1,2,4 in order to test runtime bootstrap code,
# creation of first goroutines and first garbage collections in the parallel setting.
# TODO(bradfitz,0intro): make this not be 15 minutes once Plan 9 runs quickly on GCE.
# Linux does 63 seconds on GCE, but Plan 9 was failing with 8 minutes.
# See issue 8393 and 9491.
# TODO(bradfitz,0intro): remove -v once we see some a successful build.
echo '# GOMAXPROCS=2 runtime -cpu=1,2,4'
GOMAXPROCS=2 go test runtime -v -short -timeout 15m -cpu 1,2,4
GOMAXPROCS=2 go test runtime -short -timeout 240s -cpu 1,2,4
echo
echo '# sync -cpu=10'
Expand Down

0 comments on commit c62b003

Please sign in to comment.