migrations: fix azure AdoptResources #6961

Merged
merged 2 commits into from Feb 10, 2017

Conversation

Projects
None yet
3 participants
Member

babbageclunk commented Feb 9, 2017

Description of change

Update the resource group controller tag as well as the resources.

Previously it would update all the resources but not the group, which meant the
contained resources will still be killed by an old controller being destroyed.

QA steps

  • bootstrap two controllers on Azure, A and B
  • add a model on A
  • deploy an application the model
  • migrate the model to B
  • once the migration is complete and the model is running on B, destroy controller A
  • the model should continue running on B

Bug reference

Part of fix for https://bugs.launchpad.net/juju/+bug/1648063

babbageclunk added some commits Feb 9, 2017

Also update the resource group controller tag
Otherwise the contained resources will still be killed by an old
controller being destroyed.
Strip group ProvisioningState before updating
The Azure API forbids it from being specified.

axw approved these changes Feb 10, 2017

Member

babbageclunk commented Feb 10, 2017

$$merge$$

Contributor

jujubot commented Feb 10, 2017

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

@jujubot jujubot merged commit 11a1da3 into juju:2.1 Feb 10, 2017

@babbageclunk babbageclunk deleted the babbageclunk:mm-fix-azure-adopt branch Feb 10, 2017

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