worker/lease: NewDeadManager returns manager that crashes

Bug #1686720 reported by Roger Peppe
14
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Canonical Juju
Fix Released
Critical
Eric Claude Jones

Bug Description

NewDeadManager is documented as returning a manager that returns an error
on all its operations, but actually the Claim and Token methods will panic
if called. This panic has been seen here http://juju-ci.vapour.ws:8080/job/github-merge-juju/10774/artifact/artifacts/xenial.log and here http://juju-ci.vapour.ws:8080/job/github-merge-juju/10722/artifact/artifacts/xenial.log.

To fix, perhaps initialize the Manager with a dumb Secretary instance that returns
an error from all operations.

Changed in juju:
status: New → Triaged
importance: Undecided → Critical
milestone: none → 2.2-beta4
Changed in juju:
milestone: 2.2-beta4 → 2.2-rc1
Ian Booth (wallyworld)
Changed in juju:
assignee: nobody → Eric Claude Jones (ecjones)
Changed in juju:
status: Triaged → Fix Committed
Revision history for this message
Eric Claude Jones (ecjones) wrote :
Changed in juju:
status: Fix Committed → Fix Released
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.