/juju

Revert allocation deletion on model destroy #7147

Merged
merged 1 commit into from Mar 23, 2017

Conversation

Reviewers

@wallyworld wallyworld

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
3 participants
@cmars @jujubot @wallyworld
@cmars
Owner

cmars commented Mar 23, 2017 •
Edited 1 time

Fixes LP:#1675214

This is a temporary fix that addresses the regression. For 2.2, we'll
followup with a more robust solution to allocation deletion on model
destroy.

Description of change

Why is this change needed?

Fixes LP:#1675214

QA steps

How do we verify that the change works?

The failing test linked in that bug should now pass.

Documentation changes

Does it affect current user workflow? CLI? API?

No.

Bug reference

Does this change fix a bug? Please add a link to it.

https://bugs.launchpad.net/juju/+bug/1675214
@cmars
Revert allocation deletion on model destroy 
Fixes LP:#1675214

This is a temporary fix that addresses the regression. For 2.2, we'll
followup with a more robust solution to allocation deletion on model
destroy.
6cb4e95
@cmars
Owner

cmars commented Mar 23, 2017

!!test!!
@cmars
Owner

cmars commented Mar 23, 2017

$$merge$$, check-merge failure was just some mongo connection flakiness.
@jujubot
Contributor

jujubot commented Mar 23, 2017

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

@jujubot jujubot merged commit 0a53856 into juju:rising-sun Mar 23, 2017
1 check failed

1 check failed

github-check-merge-juju Built PR, ran unit tests, and tested LXD deploy. Use !!.*!! to request another build. IE, !!build!!, !!retry!!
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment