/juju

Allow state.Space and Subnet with duplicate ProviderIds across models #4354

Merged
merged 10 commits into from Feb 10, 2016

Conversation

Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
2 participants
@dimitern @jujubot
@dimitern
Contributor

dimitern commented Feb 9, 2016

Needed to allow discoverspaces to work in models other than the
controller model.

Also includes some improvements to tests.

(Review request: http://reviews.vapour.ws/r/3798/)
@dimitern
Contributor

dimitern commented Feb 10, 2016

$$merge$$
@jujubot
Contributor

jujubot commented Feb 10, 2016

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

jujubot added a commit that referenced this pull request Feb 10, 2016

@jujubot
Merge pull request #4354 from dimitern/maas-spaces-state-fix-spaces-s… 
…ubnets

Allow state.Space and Subnet with duplicate ProviderIds across models

Needed to allow discoverspaces to work in models other than the
controller model.

Also includes some improvements to tests.

(Review request: http://reviews.vapour.ws/r/3798/)
4dba9fb

@jujubot jujubot merged commit 4dba9fb into juju:maas-spaces Feb 10, 2016

@dimitern dimitern deleted the dimitern:maas-spaces-state-fix-spaces-subnets branch Feb 10, 2016

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