Skip to content

Test_Run_OneVolumeAttachAndDetachMultipleNodesWithReadWriteOnce Flake #16787

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
pravisankar opened this issue Oct 10, 2017 · 1 comment
Closed
Assignees
Labels
component/storage kind/test-flake Categorizes issue or PR as related to test flakes. priority/P1

Comments

@pravisankar
Copy link

pravisankar commented Oct 10, 2017

https://storage.googleapis.com/origin-ci-test/pr-logs/pull/16766/test_pull_request_origin_unit/4057/build-log.txt

In suite "github.com/openshift/origin/vendor/k8s.io/kubernetes/pkg/controller/volume/attachdetach/reconciler", test case "Test_Run_OneVolumeAttachAndDetachMultipleNodesWithReadWriteOnce" failed:
RUN Test_Run_OneVolumeAttachAndDetachMultipleNodesWithReadWriteOnce
FAIL: Test_Run_OneVolumeAttachAndDetachMultipleNodesWithReadWriteOnce (0.01s)
reconciler_test.go:520: Warning: Wrong NewAttacherCallCount. Expected: <1> Actual: <0>. Will retry.
reconciler_test.go:472: Expected volume "fake-plugin/volume-name" on node "node-name2" to have multiattach error

WARNING: DATA RACE
Write at 0x00c420326ea0 by goroutine 40:
runtime.mapassign()
/usr/lib/golang/src/runtime/hashmap.go:485 +0x0
github.com/openshift/origin/vendor/k8s.io/kubernetes/pkg/controller/volume/attachdetach/cache.(*desiredStateOfWorld).SetMultiAttachError()
/go/src/github.com/openshift/origin/_output/local/go/src/github.com/openshift/origin/vendor/k8s.io/kubernetes/pkg/controller/volume/attachdetach/cache/desired_state_of_world.go:346 +0x2a1
github.com/openshift/origin/vendor/k8s.io/kubernetes/pkg/controller/volume/attachdetach/reconciler.(*reconciler).attachDesiredVolumes()
github.com/openshift/origin/vendor/k8s.io/kubernetes/pkg/controller/volume/attachdetach/reconciler/_test/_obj_test/reconciler.go:274 +0xba8
github.com/openshift/origin/vendor/k8s.io/kubernetes/pkg/controller/volume/attachdetach/reconciler.(reconciler).reconcile()
github.com/openshift/origin/vendor/k8s.io/kubernetes/pkg/controller/volume/attachdetach/reconciler/_test/_obj_test/reconciler.go:231 +0x23b
github.com/openshift/origin/vendor/k8s.io/kubernetes/pkg/controller/volume/attachdetach/reconciler.(reconciler).reconciliationLoopFunc.func1()
github.com/openshift/origin/vendor/k8s.io/kubernetes/pkg/controller/volume/attachdetach/reconciler/_test/_obj_test/reconciler.go:73 +0x78
github.com/openshift/origin/vendor/k8s.io/apimachinery/pkg/util/wait.JitterUntil.func1()

/go/src/github.com/openshift/origin/_output/local/go/src/github.com/openshift/origin/vendor/k8s.io/apimachinery/pkg/util/wait/wait.go:97 +0x6f
github.com/openshift/origin/vendor/k8s.io/apimachinery/pkg/util/wait.JitterUntil()
/go/src/github.com/openshift/origin/_output/local/go/src/github.com/openshift/origin/vendor/k8s.io/apimachinery/pkg/util/wait/wait.go:98 +0xcd
github.com/openshift/origin/vendor/k8s.io/apimachinery/pkg/util/wait.Until()
/go/src/github.com/openshift/origin/_output/local/go/src/github.com/openshift/origin/vendor/k8s.io/apimachinery/pkg/util/wait/wait.go:52 +0x5a
github.com/openshift/origin/vendor/k8s.io/kubernetes/pkg/controller/volume/attachdetach/reconciler.(*reconciler).Run()
github.com/openshift/origin/vendor/k8s.io/kubernetes/pkg/controller/volume/attachdetach/reconciler/_test/_obj_test/reconciler.go:65 +0x94
Previous read at 0x00c420326ea0 by goroutine 39:
runtime.mapiterinit()
/usr/lib/golang/src/runtime/hashmap.go:668 +0x0
github.com/openshift/origin/vendor/k8s.io/kubernetes/pkg/controller/volume/attachdetach/cache.(*desiredStateOfWorld).GetVolumesToAttach()
/go/src/github.com/openshift/origin/_output/local/go/src/github.com/openshift/origin/vendor/k8s.io/kubernetes/pkg/controller/volume/attachdetach/cache/desired_state_of_world.go:372 +0x26d
github.com/openshift/origin/vendor/k8s.io/kubernetes/pkg/controller/volume/attachdetach/reconciler.Test_Run_OneVolumeAttachAndDetachMultipleNodesWithReadWriteOnce()
/go/src/github.com/openshift/origin/_output/local/go/src/github.com/openshift/origin/vendor/k8s.io/kubernetes/pkg/controller/volume/attachdetach/reconciler/reconciler_test.go:469 +0x124c
testing.tRunner()
/usr/lib/golang/src/testing/testing.go:657 +0x107
Goroutine 40 (running) created at:
github.com/openshift/origin/vendor/k8s.io/kubernetes/pkg/controller/volume/attachdetach/reconciler.Test_Run_OneVolumeAttachAndDetachMultipleNodesWithReadWriteOnce()
/go/src/github.com/openshift/origin/_output/local/go/src/github.com/openshift/origin/vendor/k8s.io/kubernetes/pkg/controller/volume/attachdetach/reconciler/reconciler_test.go:449 +0x10bf
testing.tRunner()
/usr/lib/golang/src/testing/testing.go:657 +0x107
Goroutine 39 (running) created at:
testing.(*T).Run()
/usr/lib/golang/src/testing/testing.go:697 +0x543
testing.runTests.func1()
/usr/lib/golang/src/testing/testing.go:882 +0xaa
testing.tRunner()
/usr/lib/golang/src/testing/testing.go:657 +0x107
testing.runTests()
/usr/lib/golang/src/testing/testing.go:888 +0x4e0
testing.(*M).Run()
/usr/lib/golang/src/testing/testing.go:822 +0x1c3
main.main()
github.com/openshift/origin/vendor/k8s.io/kubernetes/pkg/controller/volume/attachdetach/reconciler/_test/_testmain.go:100 +0x33d

==================
WARNING: DATA RACE
Write at 0x00c42039c808 by goroutine 40:
github.com/openshift/origin/vendor/k8s.io/kubernetes/pkg/controller/volume/attachdetach/cache.(*desiredStateOfWorld).SetMultiAttachError()
/go/src/github.com/openshift/origin/_output/local/go/src/github.com/openshift/origin/vendor/k8s.io/kubernetes/pkg/controller/volume/attachdetach/cache/desired_state_of_world.go:346 +0x2c0
github.com/openshift/origin/vendor/k8s.io/kubernetes/pkg/controller/volume/attachdetach/reconciler.(*reconciler).attachDesiredVolumes()
github.com/openshift/origin/vendor/k8s.io/kubernetes/pkg/controller/volume/attachdetach/reconciler/_test/_obj_test/reconciler.go:274 +0xba8
github.com/openshift/origin/vendor/k8s.io/kubernetes/pkg/controller/volume/attachdetach/reconciler.(*reconciler).reconcile()
github.com/openshift/origin/vendor/k8s.io/kubernetes/pkg/controller/volume/attachdetach/reconciler/_test/_obj_test/reconciler.go:231 +0x23b
github.com/openshift/origin/vendor/k8s.io/kubernetes/pkg/controller/volume/attachdetach/reconciler.(*reconciler).reconciliationLoopFunc.func1()
github.com/openshift/origin/vendor/k8s.io/kubernetes/pkg/controller/volume/attachdetach/reconciler/_test/_obj_test/reconciler.go:73 +0x78
github.com/openshift/origin/vendor/k8s.io/apimachinery/pkg/util/wait.JitterUntil.func1()
/go/src/github.com/openshift/origin/_output/local/go/src/github.com/openshift/origin/vendor/k8s.io/apimachinery/pkg/util/wait/wait.go:97 +0x6f
github.com/openshift/origin/vendor/k8s.io/apimachinery/pkg/util/wait.JitterUntil()
/go/src/github.com/openshift/origin/_output/local/go/src/github.com/openshift/origin/vendor/k8s.io/apimachinery/pkg/util/wait/wait.go:98 +0xcd
github.com/openshift/origin/vendor/k8s.io/apimachinery/pkg/util/wait.Until()
/go/src/github.com/openshift/origin/_output/local/go/src/github.com/openshift/origin/vendor/k8s.io/apimachinery/pkg/util/wait/wait.go:52 +0x5a
github.com/openshift/origin/vendor/k8s.io/kubernetes/pkg/controller/volume/attachdetach/reconciler.(*reconciler).Run()
github.com/openshift/origin/vendor/k8s.io/kubernetes/pkg/controller/volume/attachdetach/reconciler/_test/_obj_test/reconciler.go:65 +0x94
Previous read at 0x00c42039c808 by goroutine 39:
github.com/openshift/origin/vendor/k8s.io/kubernetes/pkg/controller/volume/attachdetach/cache.(*desiredStateOfWorld).GetVolumesToAttach()
/go/src/github.com/openshift/origin/_output/local/go/src/github.com/openshift/origin/vendor/k8s.io/kubernetes/pkg/controller/volume/attachdetach/cache/desired_state_of_world.go:372 +0x2bc
github.com/openshift/origin/vendor/k8s.io/kubernetes/pkg/controller/volume/attachdetach/reconciler.Test_Run_OneVolumeAttachAndDetachMultipleNodesWithReadWriteOnce()
/go/src/github.com/openshift/origin/_output/local/go/src/github.com/openshift/origin/vendor/k8s.io/kubernetes/pkg/controller/volume/attachdetach/reconciler/reconciler_test.go:469 +0x124c
testing.tRunner()
/usr/lib/golang/src/testing/testing.go:657 +0x107
Goroutine 40 (running) created at:
github.com/openshift/origin/vendor/k8s.io/kubernetes/pkg/controller/volume/attachdetach/reconciler.Test_Run_OneVolumeAttachAndDetachMultipleNodesWithReadWriteOnce()
/go/src/github.com/openshift/origin/_output/local/go/src/github.com/openshift/origin/vendor/k8s.io/kubernetes/pkg/controller/volume/attachdetach/reconciler/reconciler_test.go:449 +0x10bf
testing.tRunner()
/usr/lib/golang/src/testing/testing.go:657 +0x107
Goroutine 39 (finished) created at:
testing.(*T).Run()
/usr/lib/golang/src/testing/testing.go:697 +0x543
testing.runTests.func1()
/usr/lib/golang/src/testing/testing.go:882 +0xaa
testing.tRunner()
/usr/lib/golang/src/testing/testing.go:657 +0x107
testing.runTests()
/usr/lib/golang/src/testing/testing.go:888 +0x4e0
testing.(*M).Run()
/usr/lib/golang/src/testing/testing.go:822 +0x1c3
main.main()
github.com/openshift/origin/vendor/k8s.io/kubernetes/pkg/controller/volume/attachdetach/reconciler/_test/_testmain.go:100 +0x33d

FAIL

@pravisankar pravisankar added kind/test-flake Categorizes issue or PR as related to test flakes. component/storage labels Oct 10, 2017
@gnufied
Copy link
Member

gnufied commented Oct 10, 2017

is this same as kubernetes/kubernetes#53590 ?

@gnufied gnufied self-assigned this Oct 10, 2017
openshift-merge-robot added a commit that referenced this issue Oct 11, 2017
Automatic merge from submit-queue.

UPSTREAM: 53682: Use proper locks when updating desired state

Fixes #16787
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component/storage kind/test-flake Categorizes issue or PR as related to test flakes. priority/P1
Projects
None yet
Development

No branches or pull requests

3 participants