/1.25-upgrade

Assorted fixes from testing with IS #50

Merged
merged 5 commits into from Oct 5, 2017

Conversation

Reviewers

@howbazaar howbazaar

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
2 participants
@babbageclunk @howbazaar
@babbageclunk
Member

babbageclunk commented Oct 5, 2017

If we find a status record with a nil updated time, use now.

Ensure that UUID is set in the environment config - it's needed in import and tag upgrading.

(Follows on from #49 - ignore first two commits.)

babbageclunk added some commits Oct 3, 2017

@babbageclunk
Update group deletion to work with controller UUID 
The workaround for the problem with AdoptResources means that security
groups that need to be deleted now are named
juju-<controlleruuid>-<modeluuid>-0, so handle that in the downgrade
step. Also remove the groups from any instances before trying to delete
them.
9c2265e
@babbageclunk
Restrict status history items to 20 for each key 
Having them unlimited was causing OOMs when serialising the model.
c0eb739
@babbageclunk
If updated is nil on a status record use now
c21e5f4
@babbageclunk
Ensure environ UUID is set in exported model config
a2e88a9
@babbageclunk
Ensure that UUID is set in the config 
It's needed for upgrading tags and potenitally other things.
84ab58e

@babbageclunk babbageclunk referenced this pull request Oct 5, 2017

Merged

Rename LXC containers to LXD during upgrade #51

@babbageclunk babbageclunk merged commit 4d6478d into juju:master Oct 5, 2017

@babbageclunk babbageclunk deleted the babbageclunk:default-updated branch Oct 5, 2017

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