Join GitHub today
GitHub is home to over 20 million developers working together to host and review code, manage projects, and build software together.
state: use errors.Cause in IsNotAssigned #1789
Conversation
|
$$fixes-1430049$$ |
|
Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju |
added a commit
that referenced
this pull request
Mar 10, 2015
jujubot
merged commit c2d16fc
into
juju:1.22
Mar 10, 2015
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
axw commentedMar 10, 2015
There was a change in PR 1707 that causes the
megawatcher to fail if the unit is not yet assigned to
a machine. The cause is that a call to errors.Cause was
removed in back-porting a fix from master, but
IsNotAssigned does not do this in 1.22 as it does on
master.
This branch changes IsNotAssigned to call errors.Cause.
Fixes https://bugs.launchpad.net/juju-core/+bug/1430049
(Review request: http://reviews.vapour.ws/r/1117/)