Code review comment for lp:~frankban/juju-quickstart/local-provider

Revision history for this message
Francesco Banconi (frankban) wrote :

Reviewers: mp+195952_code.launchpad.net,

Message:
Please take a look.

Description:
Add support for local providers.

Bootstrap the environment with "sudo" if
the environment is configured to use the
local provider.

Also improved debugging documentation.

Tests: make check

To QA this, just run `.venv/bin/python juju-quickstart`
as you are used to, but with the local provider:
try to deploy a bundle, try to re-run quickstart again
with the environment already bootstrapped.
In general the application should ask for sudo password
and then proceed as usual.

https://code.launchpad.net/~frankban/juju-quickstart/local-provider/+merge/195952

(do not edit description out of merge proposal)

Please review this at https://codereview.appspot.com/29540044/

Affected files (+133, -39 lines):
   M HACKING.rst
   A [revision details]
   M quickstart/app.py
   M quickstart/manage.py
   M quickstart/tests/test_app.py
   M quickstart/tests/test_manage.py
   M quickstart/utils.py

« Back to merge proposal