provider/joyent: multiple data races

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

Bug Description

==================
WARNING: DATA RACE
Read by goroutine 405:
  runtime.mapaccess2_faststr()
      /home/dfc/go/src/runtime/hashmap_fast.go:281 +0x0
  github.com/joyent/gomanta/localservices/manta.(*Manta).handleStorage()
      /home/dfc/src/github.com/joyent/gomanta/localservices/manta/service_http.go:234 +0x17e6
  github.com/joyent/gomanta/localservices/manta.(*mantaHandler).ServeHTTP()
      /home/dfc/src/github.com/joyent/gomanta/localservices/manta/service_http.go:98 +0x1bf
  net/http.(*ServeMux).ServeHTTP()
      /home/dfc/go/src/net/http/server.go:1596 +0x215
  net/http.serverHandler.ServeHTTP()
      /home/dfc/go/src/net/http/server.go:1758 +0x209
  net/http.(*conn).serve()
      /home/dfc/go/src/net/http/server.go:1259 +0x1181

Previous write by goroutine 607:
  runtime.mapdelete()
      /home/dfc/go/src/runtime/hashmap.go:511 +0x0
  github.com/joyent/gomanta/localservices/manta.(*Manta).DeleteObject()
      /home/dfc/src/github.com/joyent/gomanta/localservices/manta/service.go:277 +0x4f2
  github.com/joyent/gomanta/localservices/manta.(*Manta).handleStorage()
      /home/dfc/src/github.com/joyent/gomanta/localservices/manta/service_http.go:236 +0x181f
  github.com/joyent/gomanta/localservices/manta.(*mantaHandler).ServeHTTP()
      /home/dfc/src/github.com/joyent/gomanta/localservices/manta/service_http.go:98 +0x1bf
  net/http.(*ServeMux).ServeHTTP()
      /home/dfc/go/src/net/http/server.go:1596 +0x215
  net/http.serverHandler.ServeHTTP()
      /home/dfc/go/src/net/http/server.go:1758 +0x209
  net/http.(*conn).serve()
      /home/dfc/go/src/net/http/server.go:1259 +0x1181

Goroutine 405 (running) created at:
  net/http.(*Server).Serve()
      /home/dfc/go/src/net/http/server.go:1806 +0x46d

Goroutine 607 (finished) created at:
  net/http.(*Server).Serve()
      /home/dfc/go/src/net/http/server.go:1806 +0x46d
==================
==================
WARNING: DATA RACE
Read by goroutine 405:
  runtime.mapaccess2_faststr()
      /home/dfc/go/src/runtime/hashmap_fast.go:281 +0x0
  github.com/joyent/gomanta/localservices/manta.(*Manta).DeleteObject()
      /home/dfc/src/github.com/joyent/gomanta/localservices/manta/service.go:275 +0x428
  github.com/joyent/gomanta/localservices/manta.(*Manta).handleStorage()
      /home/dfc/src/github.com/joyent/gomanta/localservices/manta/service_http.go:236 +0x181f
  github.com/joyent/gomanta/localservices/manta.(*mantaHandler).ServeHTTP()
      /home/dfc/src/github.com/joyent/gomanta/localservices/manta/service_http.go:98 +0x1bf
  net/http.(*ServeMux).ServeHTTP()
      /home/dfc/go/src/net/http/server.go:1596 +0x215
  net/http.serverHandler.ServeHTTP()
      /home/dfc/go/src/net/http/server.go:1758 +0x209
  net/http.(*conn).serve()
      /home/dfc/go/src/net/http/server.go:1259 +0x1181

Previous write by goroutine 607:
  runtime.mapdelete()
      /home/dfc/go/src/runtime/hashmap.go:511 +0x0
  github.com/joyent/gomanta/localservices/manta.(*Manta).DeleteObject()
      /home/dfc/src/github.com/joyent/gomanta/localservices/manta/service.go:276 +0x494
  github.com/joyent/gomanta/localservices/manta.(*Manta).handleStorage()
      /home/dfc/src/github.com/joyent/gomanta/localservices/manta/service_http.go:236 +0x181f
  github.com/joyent/gomanta/localservices/manta.(*mantaHandler).ServeHTTP()
      /home/dfc/src/github.com/joyent/gomanta/localservices/manta/service_http.go:98 +0x1bf
  net/http.(*ServeMux).ServeHTTP()
      /home/dfc/go/src/net/http/server.go:1596 +0x215
  net/http.serverHandler.ServeHTTP()
      /home/dfc/go/src/net/http/server.go:1758 +0x209
  net/http.(*conn).serve()
      /home/dfc/go/src/net/http/server.go:1259 +0x1181

Goroutine 405 (running) created at:
  net/http.(*Server).Serve()
      /home/dfc/go/src/net/http/server.go:1806 +0x46d

Goroutine 607 (finished) created at:
  net/http.(*Server).Serve()
      /home/dfc/go/src/net/http/server.go:1806 +0x46d
==================
OK: 51 passed, 2 skipped
PASS
Found 2 data race(s)
FAIL github.com/juju/juju/provider/joyent 24.267s

Changed in juju-core:
importance: Undecided → High
assignee: nobody → Dave Cheney (dave-cheney)
Curtis Hovey (sinzui)
Changed in juju-core:
status: New → Triaged
milestone: none → 1.25.0
Curtis Hovey (sinzui)
Changed in juju-core:
status: Triaged → Incomplete
Curtis Hovey (sinzui)
Changed in juju-core:
milestone: 1.25-alpha1 → 1.25-beta1
Curtis Hovey (sinzui)
Changed in juju-core:
milestone: 1.25-beta1 → 1.25-beta2
Changed in juju-core:
milestone: 1.25-beta2 → 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
Curtis Hovey (sinzui)
Changed in juju-core:
milestone: 2.0-alpha2 → 2.0-beta1
Curtis Hovey (sinzui)
Changed in juju-core:
milestone: 2.0-beta1 → 2.0-beta2
Curtis Hovey (sinzui)
Changed in juju-core:
milestone: 2.0-beta2 → 2.0-beta3
Curtis Hovey (sinzui)
Changed in juju-core:
status: Incomplete → Triaged
Curtis Hovey (sinzui)
Changed in juju-core:
milestone: 2.0-beta3 → 2.0-beta4
Changed in juju-core:
milestone: 2.0-beta4 → 2.0.0
Curtis Hovey (sinzui)
tags: added: race-condition
Changed in juju-core:
status: Triaged → Fix Committed
Changed in juju-core:
milestone: 2.0.0 → 2.0-rc1
Curtis Hovey (sinzui)
tags: added: tech-debt
Curtis Hovey (sinzui)
Changed in juju-core:
status: Fix Committed → Fix Released
affects: juju-core → juju
Changed in juju:
milestone: 2.0-beta5 → none
milestone: none → 2.0-beta5
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

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