/juju

Fix unit cleanup #6597

Merged
merged 1 commit into from Nov 23, 2016

Conversation

Reviewers

@mjs mjs

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
3 participants
@howbazaar @jujubot @mjs
@howbazaar
Owner

howbazaar commented Nov 23, 2016

If there were completed actions for a unit, the cleanups would fail with a transaction assertion.

Fixes http://pad.lv/1640608

@howbazaar howbazaar changed the base branch from staging to develop Nov 23, 2016

@mjs

mjs approved these changes Nov 23, 2016
View changes

Thanks!

@howbazaar
Owner

howbazaar commented Nov 23, 2016

$$merge$$
@jujubot
Contributor

jujubot commented Nov 23, 2016

Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju
@jujubot
Contributor

jujubot commented Nov 23, 2016

Build failed: Tests failed
build url: http://juju-ci.vapour.ws:8080/job/github-merge-juju/9691
@howbazaar
Owner

howbazaar commented Nov 23, 2016

Can't see where that one failed. $$merge$$
@jujubot
Contributor

jujubot commented Nov 23, 2016

Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju

@jujubot jujubot merged commit d9d736e into juju:develop Nov 23, 2016

@howbazaar howbazaar deleted the howbazaar:fix-unit-cleanup branch Nov 28, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment