race in caasoperator test suite

Bug #1749004 reported by Tim Penhey on 2018-02-12
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
juju
High
Ian Booth

Bug Description

        ==================
WARNING: DATA RACE
Read at 0x00c4200a7e90 by goroutine 84:
  github.com/juju/juju/worker/caasoperator_test.(*WorkerSuite).TestWorkerDownloadsCharm()
      /workspace/src/github.com/juju/juju/worker/caasoperator/caasoperator_test.go:235 +0x15da
  runtime.call32()
      /snap/go/1404/src/runtime/asm_amd64.s:509 +0x3a
  reflect.Value.Call()
      /snap/go/1404/src/reflect/value.go:302 +0xc0
  gopkg.in/check%2ev1.(*suiteRunner).forkTest.func1()
      /workspace/src/gopkg.in/check.v1/check.go:772 +0x9ab
  gopkg.in/check%2ev1.(*suiteRunner).forkCall.func1()
      /workspace/src/gopkg.in/check.v1/check.go:666 +0x89

Previous write at 0x00c4200a7e90 by goroutine 91:
  github.com/juju/juju/worker/caasoperator_test.(*WorkerSuite).TestWorkerDownloadsCharm.func1()
      /workspace/src/github.com/juju/juju/worker/caasoperator/caasoperator_test.go:185 +0x169
  github.com/juju/juju/worker/caasoperator.(*caasOperator).loop()
      /workspace/src/github.com/juju/juju/worker/caasoperator/caasoperator.go:285 +0x824
  github.com/juju/juju/worker/caasoperator.(*caasOperator).(github.com/juju/juju/worker/caasoperator.loop)-fm()
      /workspace/src/github.com/juju/juju/worker/caasoperator/caasoperator.go:180 +0x41
  github.com/juju/juju/worker/catacomb.runSafely()
      /workspace/src/github.com/juju/juju/worker/catacomb/catacomb.go:289 +0x70
  github.com/juju/juju/worker/catacomb.Invoke.func3()
      /workspace/src/github.com/juju/juju/worker/catacomb/catacomb.go:116 +0x8e

Goroutine 84 (running) created at:
  gopkg.in/check%2ev1.(*suiteRunner).forkCall()
      /workspace/src/gopkg.in/check.v1/check.go:663 +0x430
  gopkg.in/check%2ev1.(*suiteRunner).forkTest()
      /workspace/src/gopkg.in/check.v1/check.go:754 +0x131
  gopkg.in/check%2ev1.(*suiteRunner).runTest()
      /workspace/src/gopkg.in/check.v1/check.go:809 +0x42
  gopkg.in/check%2ev1.(*suiteRunner).run()
      /workspace/src/gopkg.in/check.v1/check.go:615 +0x1fa
  gopkg.in/check%2ev1.Run()
      /workspace/src/gopkg.in/check.v1/run.go:92 +0x5a
  gopkg.in/check%2ev1.RunAll()
      /workspace/src/gopkg.in/check.v1/run.go:84 +0x136
  gopkg.in/check%2ev1.TestingT()
      /workspace/src/gopkg.in/check.v1/run.go:72 +0x680
  github.com/juju/juju/worker/caasoperator_test.TestPackage()
      /workspace/src/github.com/juju/juju/worker/caasoperator/package_test.go:13 +0x38
  testing.tRunner()
      /snap/go/1404/src/testing/testing.go:746 +0x16c

Goroutine 91 (running) created at:
  github.com/juju/juju/worker/catacomb.Invoke()
      /workspace/src/github.com/juju/juju/worker/catacomb/catacomb.go:113 +0x286
  github.com/juju/juju/worker/caasoperator.NewWorker()
      /workspace/src/github.com/juju/juju/worker/caasoperator/caasoperator.go:178 +0x477
  github.com/juju/juju/worker/caasoperator_test.(*WorkerSuite).TestWorkerDownloadsCharm()
      /workspace/src/github.com/juju/juju/worker/caasoperator/caasoperator_test.go:189 +0x1b3
  runtime.call32()
      /snap/go/1404/src/runtime/asm_amd64.s:509 +0x3a
  reflect.Value.Call()
      /snap/go/1404/src/reflect/value.go:302 +0xc0
  gopkg.in/check%2ev1.(*suiteRunner).forkTest.func1()
      /workspace/src/gopkg.in/check.v1/check.go:772 +0x9ab
  gopkg.in/check%2ev1.(*suiteRunner).forkCall.func1()
      /workspace/src/gopkg.in/check.v1/check.go:666 +0x89
==================
OK: 10 passed
 testing.go:699: race detected during execution of test

Ian Booth (wallyworld) on 2018-02-13
Changed in juju:
assignee: nobody → Ian Booth (wallyworld)
status: Triaged → In Progress
Ian Booth (wallyworld) on 2018-02-14
Changed in juju:
status: In Progress → Fix Committed
Ian Booth (wallyworld) on 2018-02-14
Changed in juju:
milestone: none → 2.4-beta1
Changed in juju:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers