race in caasoperator test suite

Bug #1749004 reported by Tim Penhey
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Canonical Juju
Fix Released
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)
Changed in juju:
assignee: nobody → Ian Booth (wallyworld)
status: Triaged → In Progress
Ian Booth (wallyworld)
Changed in juju:
status: In Progress → Fix Committed
Ian Booth (wallyworld)
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  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.