Skip to content

/go

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

net: TestWriteTimeoutFluctuation sometimes fails on 32-bit iOS builder #10775

Open
mikioh opened this Issue May 11, 2015 · 1 comment

Comments

Assignees
No one assigned
Labels
OS-Darwin
Projects
None yet
Milestone
  Unplanned
2 participants
@mikioh @gopherbot
@mikioh
Copy link
Contributor

mikioh commented May 11, 2015

See build dashboard.
http://build.golang.org/log/b02e6aaa2fe497fadf898192d5425e7201569703
http://build.golang.org/log/350de48b4c0e31390ac02f7a1af96c22401a5628
http://build.golang.org/log/58502b25b5c6dc158954ba111690223cb5bc078f
http://build.golang.org/log/2bb42bac80498052be38e335298d05b43d370f1a

No errors at the package API, looks like timeoutTransmitter goroutine is not scheduled well, not sure the reason but seems like it needs further investigation.

process continue
Process 580 resuming
--- FAIL: TestWriteTimeoutFluctuation (1.89s)

    timeout_test.go:706: Write took over 1s; expected 0.1s

FAIL

Process 580 exited with status = 1 (0x00000001) 
go_darwin_arm_exec: timeout running tests

@mikioh mikioh added the OS-Darwin label May 11, 2015

@mikioh mikioh added this to the Unplanned milestone May 11, 2015

@gopherbot

This comment has been minimized.

Sign in to view
Copy link

gopherbot commented May 11, 2015

CL https://golang.org/cl/9915 mentions this issue.

mk0x9 pushed a commit to mk0x9/go that referenced this issue May 11, 2015

Mikio Hara
net: increase timeout in TestWriteTimeoutFluctuation on darwin/arm 
On darwin/arm, the test sometimes fails with:

Process 557 resuming
--- FAIL: TestWriteTimeoutFluctuation (1.64s)
	timeout_test.go:706: Write took over 1s; expected 0.1s
FAIL
Process 557 exited with status = 1 (0x00000001)
go_darwin_arm_exec: timeout running tests

This change increaes timeout on iOS builders from 1s to 3s as a
temporarily fix.

Updates golang#10775.

Change-Id: Ifdaf99cf5b8582c1a636a0f7d5cc66bb276efd72
Reviewed-on: https://go-review.googlesource.com/9915
Reviewed-by: Minux Ma <minux@golang.org>
cbcc758
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.