/juju

apiserver/modelmanager: Removed ForModel call in DumpModel #7524

Merged
merged 1 commit into from Jun 20, 2017

Conversation

Reviewers

@babbageclunk babbageclunk

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
3 participants
@mjs @jujubot @babbageclunk
@mjs
Contributor

mjs commented Jun 20, 2017 •
Edited 1 time

Description of change

Use the StatePool instead as this is more efficient.

QA steps

Bootstrap, create several models and ensure that juju dump-model works.

Documentation changes

N.A.

Bug reference

Part of the fix for https://bugs.launchpad.net/juju/2.2/+bug/1698702
@mjs
apiserver/modelmanager: Removed ForModel call in DumpModel 
Use the StatePool instead as this is more efficient.

Part of the fix for https://bugs.launchpad.net/juju/2.2/+bug/1698702
076789f
@babbageclunk

babbageclunk approved these changes Jun 20, 2017
View changes

Much better.

@mjs
Contributor

mjs commented Jun 20, 2017

$$merge$$
@jujubot
Contributor

jujubot commented Jun 20, 2017

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

@jujubot jujubot merged commit c301453 into juju:2.2 Jun 20, 2017
1 check passed

1 check passed

github-check-merge-juju Ran tests against PR. Use !!.*!! to request another build. IE, !!build!!, !!retry!!
Details

@mjs mjs deleted the mjs:1698702-dumpmodel-ForModel-2.2 branch Jun 20, 2017

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