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

Jump to bottom

crypto/tls: TestConnReadNonzeroAndEOF fails or hangs #7683

Closed
rsc opened this issue Apr 1, 2014 · 2 comments
Closed

crypto/tls: TestConnReadNonzeroAndEOF fails or hangs #7683

rsc opened this issue Apr 1, 2014 · 2 comments
Assignees
@bradfitz
Labels
FrozenDueToAge
Milestone
Go1.3

Comments

@rsc
Copy link
Contributor

@rsc rsc commented Apr 1, 2014 

go test -short crypto/tls -test.run ConnReadNonzeroAndEOF -test.cpu=$(perl -e 'print
("1,"x99)."1"')
never passes on my MacBook Pro. Typically five or so of the tests fail.

Once in a while (rarely) the test completely hangs instead.
@bradfitz

This comment has been minimized.

Sign in to view
Copy link
Contributor

@bradfitz bradfitz commented Apr 2, 2014

Comment 1:

Sent https://golang.org/cl/83080048

Status changed to Started.
@bradfitz

This comment has been minimized.

Sign in to view
Copy link
Contributor

@bradfitz bradfitz commented Apr 2, 2014

Comment 2:

This issue was closed by revision 84db9e0.

Status changed to Fixed.
@rsc rsc added fixed labels Apr 2, 2014
@rsc rsc added this to the Go1.3 milestone Apr 14, 2015
@rsc rsc removed the release-go1.3 label Apr 14, 2015
@golang golang locked and limited conversation to collaborators Jun 25, 2016
FiloSottile pushed a commit to FiloSottile/go that referenced this issue Oct 12, 2018
@bradfitz
crypto/tls: deflake TestConnReadNonzeroAndEOF
53431b9 
Fixes golang#7683

LGTM=rsc
R=rsc
CC=golang-codereviews
https://golang.org/cl/83080048
This issue was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@bradfitz bradfitz

Labels
FrozenDueToAge
Projects
None yet
Milestone
  Go1.3
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
3 participants
@rsc @bradfitz @gopherbot
You can’t perform that action at this time.