Skip to content

os/exec: TestExtraFiles fails #7531

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

Closed
gpaul opened this issue Mar 13, 2014 · 3 comments
Closed

os/exec: TestExtraFiles fails #7531

gpaul opened this issue Mar 13, 2014 · 3 comments

Comments

@gpaul
Copy link
Contributor

gpaul commented Mar 13, 2014

go version devel +d8c952ef0940 Thu Mar 13 18:26:01 2014 +1100 linux/amd64

What steps reproduce the problem?

I saw this while looping run.bash --no-rebuild with random GOMAXPROCS

...
=== RUN TestExtraFiles-8
2014/03/13 07:52:52 http: TLS handshake error from 127.0.0.1:56379: remote error: bad
certificate
--- FAIL: TestExtraFiles-8 (0.08 seconds)
        exec_test.go:434: Run: exit status 1; stdout "leaked parent file. fd = 19; want 18\n", stderr ""
=== RUN TestExtraFilesRace-8
--- PASS: TestExtraFilesRace-8 (0.37 seconds)
=== RUN TestHelperProcess-8
--- PASS: TestHelperProcess-8 (0.00 seconds)
FAIL
FAIL    os/exec 0.576s
@gpaul
Copy link
Contributor Author

gpaul commented Mar 13, 2014

Comment 1:

The following reproduces the issue after a couple of loops.
for i in `seq 1 5000` ; do  export GOMAXPROCS=$[ 1 + $[ RANDOM % 128 ] ]; echo
GOMAXPROCS=$GOMAXPROCS; go test -v -short os/exec; done 2>&1 | tee exec.log

@gpaul
Copy link
Contributor Author

gpaul commented Apr 15, 2014

Comment 2:

I'm no longer able to reproduce this on tip. Feel free to close.

@ianlancetaylor
Copy link
Contributor

Comment 3:

Thanks for following up.

Status changed to Fixed.

@gpaul gpaul added the fixed label Apr 15, 2014
@golang golang locked and limited conversation to collaborators Jun 25, 2016
This issue was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants