apiserver: Replace ForModel with pool in ModelStatus #7604

Merged
merged 1 commit into from Jul 7, 2017

Conversation

Projects
None yet
3 participants
Member

babbageclunk commented Jul 7, 2017

Description of change

Having status use an existing state instead of starting a fresh one (with the attendant workers) is more efficient.

QA steps

Bootstrap and create a few models. Verify that status works.

Bug reference

Part of https://bugs.launchpad.net/juju/+bug/1698702

axw approved these changes Jul 7, 2017

Member

babbageclunk commented Jul 7, 2017

$$merge$$

Contributor

jujubot commented Jul 7, 2017

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

@jujubot jujubot merged commit 0b1d497 into juju:2.2 Jul 7, 2017

1 check passed

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

babbageclunk commented Jul 7, 2017

@babbageclunk babbageclunk deleted the babbageclunk:remove-formodel branch Jul 7, 2017

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