-
Notifications
You must be signed in to change notification settings - Fork 18k
cmd/go: Stack overflow on running: go test -race with -coverpkg=all #23694
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
Comments
Hello @drakonka, thank you for the bug report! So interestingly I was able to first reproduce this issue in succession using Go 15 commits behind tip bcc86d5 where the latest Go tip is at 10d096f. However on pulling from master, rebuilding Go to ensure that it also exists on the latest, I can now no longer reproduce, even jumping back 15 commits back(where I originally was). I suspect this is something related to caching and in deed, the erroring stack trace originates from caching code in go/src/cmd/go/internal/work/action.go Lines 330 to 336 in 10d096f
/cc @rsc for some eyes and ideas @rsc perhaps you might not like this suggestion but I suspect if @drakonka clears their test cache this issue will disappear(@drakonka please don't yet clear your cache :)). @rsc might you know if we have a command for them to upload their test cache for inspection? |
This is a bug when using |
Change https://golang.org/cl/91875 mentions this issue: |
Change https://golang.org/cl/92075 mentions this issue: |
Updates #23694 Change-Id: I5fdad8cceacb8bbc85ca2661eb6482aa80343656 Reviewed-on: https://go-review.googlesource.com/92075 Run-TryBot: Ian Lance Taylor <iant@golang.org> Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
What version of Go are you using (
go version
)?go version go1.10rc1 linux/amd64
Does this issue reproduce with the latest release?
Yes, as far as I know this is the latest release
What operating system and processor architecture are you using (
go env
)?Fedora 26; GOARCH="amd64"
What did you do?
I ran the following command in the root of my project:
go test -race -coverpkg=all -coverprofile allcoverage1.10.out ./...
If possible, provide a recipe for reproducing the error.
Project source can be found here: https://gitlab.com/drakonka/gosnaillife
I am able to repro the issue 100% by cd-ing into the root dir of the project and running the command above, either through standalone terminal or through JetBrains Goland.
What did you expect to see?
I expect all tests in each package of the project to be run.
What did you see instead?
A stack overflow (full output below). Note that removing the -race arg or the -coverpkg arg avoids this issue. The tests encounter some unexpected failures I didn't see in 1.9, but at least they run instead of failing right away with the following output. This error can also be seen in my CI setup using golang:1.10-rc-stretch image: https://gitlab.com/drakonka/gosnaillife/-/jobs/50965296
The text was updated successfully, but these errors were encountered: