Join GitHub today
GitHub is home to over 20 million developers working together to host and review code, manage projects, and build software together.
state: don't use ForModel in State.ToolsStorage #7550
Conversation
|
$$merge$$ |
|
Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju |
|
Build failed: Tests failed |
|
$$merge$$ |
|
Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju |
|
Build failed: Tests failed |
|
$$merge$$ |
|
Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju |
|
Build failed: Tests failed |
|
$$merge$$ |
|
Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju |
axw commentedJun 23, 2017
Description of change
Stop using ForModel in the State.ToolsStorage method. Instead, just copy the database for the controller model UUID, avoiding starting/stopping workers unnecessarily.
QA steps
Smoke test bootstrap/add-machine.
Documentation changes
None.
Bug reference
None.