provider/joyent/gomanta: data race

Bug #1517747 reported by Dave Cheney
This bug report is a duplicate of:  Bug #1460882: provider/joyent: multiple data races. Edit Remove
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
juju-core
Triaged
High
Unassigned

Bug Description

==================
WARNING: DATA RACE
Read by goroutine 386:
  runtime.mapaccess2_faststr()
      /home/dfc/go/src/runtime/hashmap_fast.go:283 +0x0
  github.com/joyent/gomanta/localservices/manta.(*Manta).handleStorage()
      /home/dfc/src/github.com/joyent/gomanta/localservices/manta/service_http.go:234 +0x17e0
  github.com/joyent/gomanta/localservices/manta.(*mantaHandler).ServeHTTP()
      /home/dfc/src/github.com/joyent/gomanta/localservices/manta/service_http.go:98 +0x1bc
  net/http.(*ServeMux).ServeHTTP()
      /home/dfc/go/src/net/http/server.go:1718 +0x212
  net/http.serverHandler.ServeHTTP()
      /home/dfc/go/src/net/http/server.go:1883 +0x206
  net/http.(*conn).serve()
      /home/dfc/go/src/net/http/server.go:1374 +0x1243

Previous write by goroutine 343:
  runtime.mapdelete()
      /home/dfc/go/src/runtime/hashmap.go:525 +0x0
  github.com/joyent/gomanta/localservices/manta.(*Manta).DeleteObject()
      /home/dfc/src/github.com/joyent/gomanta/localservices/manta/service.go:277 +0x4e3
  github.com/joyent/gomanta/localservices/manta.(*Manta).handleStorage()
      /home/dfc/src/github.com/joyent/gomanta/localservices/manta/service_http.go:236 +0x1819
  github.com/joyent/gomanta/localservices/manta.(*mantaHandler).ServeHTTP()
      /home/dfc/src/github.com/joyent/gomanta/localservices/manta/service_http.go:98 +0x1bc
  net/http.(*ServeMux).ServeHTTP()
      /home/dfc/go/src/net/http/server.go:1718 +0x212
  net/http.serverHandler.ServeHTTP()
      /home/dfc/go/src/net/http/server.go:1883 +0x206
  net/http.(*conn).serve()
      /home/dfc/go/src/net/http/server.go:1374 +0x1243

Goroutine 386 (running) created at:
  net/http.(*Server).Serve()
      /home/dfc/go/src/net/http/server.go:1941 +0x4fb
  net/http/httptest.(*Server).goServe.func1()
      /home/dfc/go/src/net/http/httptest/server.go:216 +0xac

Goroutine 343 (finished) created at:
  net/http.(*Server).Serve()
      /home/dfc/go/src/net/http/server.go:1941 +0x4fb
  net/http/httptest.(*Server).goServe.func1()
      /home/dfc/go/src/net/http/httptest/server.go:216 +0xac
==================
==================
WARNING: DATA RACE
Read by goroutine 386:
  runtime.mapaccess2_faststr()
      /home/dfc/go/src/runtime/hashmap_fast.go:283 +0x0
  github.com/joyent/gomanta/localservices/manta.(*Manta).DeleteObject()
      /home/dfc/src/github.com/joyent/gomanta/localservices/manta/service.go:275 +0x419
  github.com/joyent/gomanta/localservices/manta.(*Manta).handleStorage()
      /home/dfc/src/github.com/joyent/gomanta/localservices/manta/service_http.go:236 +0x1819
  github.com/joyent/gomanta/localservices/manta.(*mantaHandler).ServeHTTP()
      /home/dfc/src/github.com/joyent/gomanta/localservices/manta/service_http.go:98 +0x1bc
  net/http.(*ServeMux).ServeHTTP()
      /home/dfc/go/src/net/http/server.go:1718 +0x212
  net/http.serverHandler.ServeHTTP()
      /home/dfc/go/src/net/http/server.go:1883 +0x206
  net/http.(*conn).serve()
      /home/dfc/go/src/net/http/server.go:1374 +0x1243

Previous write by goroutine 343:
  runtime.mapdelete()
      /home/dfc/go/src/runtime/hashmap.go:525 +0x0
  github.com/joyent/gomanta/localservices/manta.(*Manta).DeleteObject()
      /home/dfc/src/github.com/joyent/gomanta/localservices/manta/service.go:276 +0x485
  github.com/joyent/gomanta/localservices/manta.(*Manta).handleStorage()
      /home/dfc/src/github.com/joyent/gomanta/localservices/manta/service_http.go:236 +0x1819
  github.com/joyent/gomanta/localservices/manta.(*mantaHandler).ServeHTTP()
      /home/dfc/src/github.com/joyent/gomanta/localservices/manta/service_http.go:98 +0x1bc
  net/http.(*ServeMux).ServeHTTP()
      /home/dfc/go/src/net/http/server.go:1718 +0x212
  net/http.serverHandler.ServeHTTP()
      /home/dfc/go/src/net/http/server.go:1883 +0x206
  net/http.(*conn).serve()
      /home/dfc/go/src/net/http/server.go:1374 +0x1243

Goroutine 386 (running) created at:
  net/http.(*Server).Serve()
      /home/dfc/go/src/net/http/server.go:1941 +0x4fb
  net/http/httptest.(*Server).goServe.func1()
      /home/dfc/go/src/net/http/httptest/server.go:216 +0xac

Goroutine 343 (finished) created at:
  net/http.(*Server).Serve()
      /home/dfc/go/src/net/http/server.go:1941 +0x4fb
  net/http/httptest.(*Server).goServe.func1()
      /home/dfc/go/src/net/http/httptest/server.go:216 +0xac
==================
OK: 53 passed, 2 skipped
PASS
Found 2 data race(s)
FAIL github.com/juju/juju/provider/joyent 61.578s

Changed in juju-core:
status: New → Triaged
importance: Undecided → High
Curtis Hovey (sinzui)
summary: - provider/joyen: more data races
+ provider/joyent/gomanta: data race
tags: added: 2.0-count
Curtis Hovey (sinzui)
tags: added: race-condition
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.