juju

provider/maas: race in launchpad.net/gomaasapi

Bug #1468972 reported by Dave Cheney on 2015-06-26
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
​ juju
Fix Released
Medium
Dave Cheney
juju 2.0-alpha2
Also affects project (?) Also affects distribution/package Nominate for series

Bug Description

==================
WARNING: DATA RACE
Write by goroutine 64:
  runtime.mapdelete()
      /home/dfc/go/src/runtime/hashmap.go:511 +0x0
  launchpad.net/gomaasapi.fileHandler()
      /home/dfc/src/launchpad.net/gomaasapi/testservice.go:889 +0x10b
  launchpad.net/gomaasapi.filesHandler()
      /home/dfc/src/launchpad.net/gomaasapi/testservice.go:832 +0x7ea
  launchpad.net/gomaasapi.NewTestServer.func2()
      /home/dfc/src/launchpad.net/gomaasapi/testservice.go:477 +0x4b
  net/http.HandlerFunc.ServeHTTP()
      /home/dfc/go/src/net/http/server.go:1320 +0x47
  net/http.(*ServeMux).ServeHTTP()
      /home/dfc/go/src/net/http/server.go:1597 +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:1759 +0x206
  net/http.(*conn).serve()
      /home/dfc/go/src/net/http/server.go:1259 +0x1178

Previous write by goroutine 186:
  runtime.mapdelete()
      /home/dfc/go/src/runtime/hashmap.go:511 +0x0
  launchpad.net/gomaasapi.fileHandler()
      /home/dfc/src/launchpad.net/gomaasapi/testservice.go:889 +0x10b
  launchpad.net/gomaasapi.filesHandler()
      /home/dfc/src/launchpad.net/gomaasapi/testservice.go:832 +0x7ea
  launchpad.net/gomaasapi.NewTestServer.func2()
      /home/dfc/src/launchpad.net/gomaasapi/testservice.go:477 +0x4b
  net/http.HandlerFunc.ServeHTTP()
      /home/dfc/go/src/net/http/server.go:1320 +0x47
  net/http.(*ServeMux).ServeHTTP()
      /home/dfc/go/src/net/http/server.go:1597 +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:1759 +0x206
  net/http.(*conn).serve()
      /home/dfc/go/src/net/http/server.go:1259 +0x1178

Goroutine 64 (running) created at:
  net/http.(*Server).Serve()
      /home/dfc/go/src/net/http/server.go:1807 +0x464

Goroutine 186 (running) created at:
  net/http.(*Server).Serve()
      /home/dfc/go/src/net/http/server.go:1807 +0x464
==================
OK: 152 passed
PASS
Found 1 data race(s)
FAIL github.com/juju/juju/provider/maas 29.745s

Curtis Hovey (sinzui) on 2015-06-29
Changed in juju-core:
status: New → Triaged
importance: Undecided → Medium
tags: added: maas-provider race-condition
Dave Cheney (dave-cheney) on 2016-01-19
Changed in juju-core:
status: Triaged → In Progress
assignee: nobody → Dave Cheney (dave-cheney)
milestone: none → 2.0-alpha1
milestone: 2.0-alpha1 → 2.0-alpha2
Cheryl Jennings (cherylj) wrote :

PR: https://github.com/juju/juju/pull/4146

Changed in juju-core:
status: In Progress → Fix Committed
Curtis Hovey (sinzui) on 2016-02-11
Changed in juju-core:
status: Fix Committed → Fix Released
Canonical Juju QA Bot (juju-qa-bot) on 2016-08-23
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  Edit
Everyone can see this information.
Subscribing...

Other bug subscribers