provider/maas: data races

Bug #1497802 reported by Dave Cheney
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Canonical Juju
Fix Released
High
Dave Cheney

Bug Description

==================
WARNING: DATA RACE
Write by goroutine 87:
  runtime.mapdelete()
      /home/dfc/go/src/runtime/hashmap.go:513 +0x0
  launchpad.net/gomaasapi.fileHandler()
      /home/dfc/src/launchpad.net/gomaasapi/testservice.go:1134 +0x10b
  launchpad.net/gomaasapi.filesHandler()
      /home/dfc/src/launchpad.net/gomaasapi/testservice.go:1077 +0x7ea
  launchpad.net/gomaasapi.NewTestServer.func3()
      /home/dfc/src/launchpad.net/gomaasapi/testservice.go:519 +0x4b
  net/http.HandlerFunc.ServeHTTP()
      /home/dfc/go/src/net/http/server.go:1433 +0x47
  net/http.(*ServeMux).ServeHTTP()
      /home/dfc/go/src/net/http/server.go:1710 +0x212
  net/http/httptest.(*waitGroupHandler).ServeHTTP()
      /home/dfc/go/src/net/http/httptest/server.go:200 +0xfe
  net/http.serverHandler.ServeHTTP()
      /home/dfc/go/src/net/http/server.go:1873 +0x206
  net/http.(*conn).serve()
      /home/dfc/go/src/net/http/server.go:1372 +0x117c

Previous write by goroutine 90:
  runtime.mapdelete()
      /home/dfc/go/src/runtime/hashmap.go:513 +0x0
  launchpad.net/gomaasapi.fileHandler()
      /home/dfc/src/launchpad.net/gomaasapi/testservice.go:1134 +0x10b
  launchpad.net/gomaasapi.filesHandler()
      /home/dfc/src/launchpad.net/gomaasapi/testservice.go:1077 +0x7ea
  launchpad.net/gomaasapi.NewTestServer.func3()
      /home/dfc/src/launchpad.net/gomaasapi/testservice.go:519 +0x4b
  net/http.HandlerFunc.ServeHTTP()
      /home/dfc/go/src/net/http/server.go:1433 +0x47
  net/http.(*ServeMux).ServeHTTP()
      /home/dfc/go/src/net/http/server.go:1710 +0x212
  net/http/httptest.(*waitGroupHandler).ServeHTTP()
      /home/dfc/go/src/net/http/httptest/server.go:200 +0xfe
  net/http.serverHandler.ServeHTTP()
      /home/dfc/go/src/net/http/server.go:1873 +0x206
  net/http.(*conn).serve()
      /home/dfc/go/src/net/http/server.go:1372 +0x117c

Goroutine 87 (running) created at:
  net/http.(*Server).Serve()
      /home/dfc/go/src/net/http/server.go:1923 +0x455

Goroutine 90 (running) created at:
  net/http.(*Server).Serve()
      /home/dfc/go/src/net/http/server.go:1923 +0x455
==================
OK: 154 passed
PASS
Found 1 data race(s)
FAIL github.com/juju/juju/provider/maas 13.986s

Aaron Bentley (abentley)
Changed in juju-core:
status: New → Triaged
importance: Undecided → High
milestone: none → 1.26-alpha1
Curtis Hovey (sinzui)
Changed in juju-core:
milestone: 1.26-alpha1 → 1.26-alpha2
Changed in juju-core:
milestone: 1.26-alpha2 → 1.26-beta1
Changed in juju-core:
milestone: 1.26-beta1 → 2.0-alpha2
Changed in juju-core:
status: Triaged → Fix Committed
status: Fix Committed → In Progress
assignee: nobody → Dave Cheney (dave-cheney)
Revision history for this message
Martin Packman (gz) wrote :

Had an actual test error from what I assume is this problem on master:

<http://reports.vapour.ws/releases/issue/569e081a749a565a7d735ed2>

Revision history for this message
Cheryl Jennings (cherylj) wrote :
Changed in juju-core:
status: In Progress → Fix Committed
Curtis Hovey (sinzui)
Changed in juju-core:
status: Fix Committed → Fix Released
affects: juju-core → juju
Changed in juju:
milestone: 2.0-alpha2 → none
milestone: none → 2.0-alpha2
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.