Race in github.com/juju/juju/worker/catacomb/catacomb

Bug #1579057 reported by Curtis Hovey
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Canonical Juju
Fix Released
Critical
William Reade

Bug Description

As seen in
    http://reports.vapour.ws/releases/issue/572c9149749a5664afc0c132

There is a race in github.com/juju/juju/worker/catacomb/catacomb.go between several worker functions.

Curtis Hovey (sinzui)
Changed in juju-core:
status: New → Triaged
importance: Undecided → Critical
milestone: none → 2.0-beta7
tags: added: blocker ci race-condition regression
William Reade (fwereade)
Changed in juju-core:
assignee: nobody → William Reade (fwereade)
Changed in juju-core:
status: Triaged → In Progress
Revision history for this message
William Reade (fwereade) wrote :

For future reference, the catacomb stuff is a red herring -- that's pure infrastructure, and it's not surprising to see it because it's useful for most workers.

The actual problem was a leaked reference to an internal map that should have stayed inside uniter/remotestate.Watcher.Snapshot where it's safely mutexed.

Currently merging https://github.com/juju/juju/pull/5356

William Reade (fwereade)
Changed in juju-core:
status: In Progress → Fix Released
status: Fix Released → Fix Committed
Curtis Hovey (sinzui)
Changed in juju-core:
status: Fix Committed → Fix Released
affects: juju-core → juju
Changed in juju:
milestone: 2.0-beta7 → none
milestone: none → 2.0-beta7
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.