MachineSuite.TestDyingModelCleanedUp state.Life dead not alive

Bug #1583772 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 at
    http://reports.vapour.ws/releases/issue/573e1557749a56565ce182d5

the MachineSuite.TestDyingModelCleanedUp test reports that state.Life is dead not alive.

[LOG] 0:04.055 DEBUG juju.worker.dependency "undertaker" manifold worker stopped: <nil>
    c.Check(model.Life(), gc.Equals, state.Dead)
... obtained state.Life = 0 ("alive")
... expected state.Life = 2 ("dead")

Curtis Hovey (sinzui)
tags: added: s390x
removed: s3
tags: added: windows
Curtis Hovey (sinzui)
Changed in juju-core:
importance: High → Critical
Changed in juju-core:
assignee: nobody → Dave Cheney (dave-cheney)
Revision history for this message
Dave Cheney (dave-cheney) wrote :
Changed in juju-core:
status: Triaged → In Progress
tags: added: blocker
Revision history for this message
William Reade (fwereade) wrote :

Note: the test is garbage, it's testing the wrong thing: someone changed undertaker behaviour a couple of weeks ago and didn't update these tests. This is a test change, not a code change, and FWIW http://reviews.vapour.ws/r/4873/ includes a drive-by fix.

William Reade (fwereade)
Changed in juju-core:
assignee: Dave Cheney (dave-cheney) → William Reade (fwereade)
William Reade (fwereade)
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-beta8 → none
milestone: none → 2.0-beta8
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.