Skip to content
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

Data race detected by travis #711

Closed
jrick opened this issue May 3, 2017 · 3 comments
Closed

Data race detected by travis #711

jrick opened this issue May 3, 2017 · 3 comments
Labels
Milestone

Comments

@jrick
Copy link
Member

jrick commented May 3, 2017

==================
WARNING: DATA RACE
Write at 0x00c420077701 by goroutine 20:
  testing.(*common).setRan()
      /home/travis/.gimme/versions/go1.8.1.linux.amd64/src/testing/testing.go:433 +0x97
  testing.(*common).setRan()
      /home/travis/.gimme/versions/go1.8.1.linux.amd64/src/testing/testing.go:429 +0xea
  testing.(*common).setRan()
      /home/travis/.gimme/versions/go1.8.1.linux.amd64/src/testing/testing.go:429 +0xea
  testing.tRunner.func1()
      /home/travis/.gimme/versions/go1.8.1.linux.amd64/src/testing/testing.go:650 +0x48c
  testing.tRunner()
      /home/travis/.gimme/versions/go1.8.1.linux.amd64/src/testing/testing.go:659 +0x123

Previous read at 0x00c420077701 by main goroutine:
  testing.runTests()
      /home/travis/.gimme/versions/go1.8.1.linux.amd64/src/testing/testing.go:890 +0x538
  testing.(*M).Run()
      /home/travis/.gimme/versions/go1.8.1.linux.amd64/src/testing/testing.go:822 +0x1c3
  main.main()
      github.com/decred/dcrwallet/wallet/udb/_test/_testmain.go:50 +0x20f

Goroutine 20 (running) created at:
  testing.(*T).Run()
      /home/travis/.gimme/versions/go1.8.1.linux.amd64/src/testing/testing.go:697 +0x543
  github.com/decred/dcrwallet/wallet/udb.TestUpgrades.func1()
      /home/travis/gopath/src/github.com/decred/dcrwallet/wallet/udb/upgrades_test.go:81 +0x17d
==================
@jrick
Copy link
Member Author

jrick commented May 3, 2017

Appears to be unrelated to any dcrwallet code that is included in the main package. Looks like either a race in the testing package or incorrect usage in a test.

@jrick jrick added the tests label May 4, 2017
@jrick
Copy link
Member Author

jrick commented May 9, 2017

Haven't seen this since and it doesn't look a defect in any of our code.

@jrick jrick closed this as completed May 9, 2017
@jrick
Copy link
Member Author

jrick commented May 13, 2017

Reopening because I have a workaround (or what should have been the code all along).

@jrick jrick reopened this May 13, 2017
@jrick jrick added this to the v1.1.0 milestone May 13, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant