juju-quickstart

Code review comment for lp:~frankban/juju-quickstart/maas-address

  1. maas-address
  2. Merge into trunk
Revision history for this message
Richard Harding (rharding) wrote :

LGTM ty for the updates!

https://codereview.appspot.com/174790043/diff/1/quickstart/app.py
File quickstart/app.py (left):

https://codereview.appspot.com/174790043/diff/1/quickstart/app.py#oldcode188
quickstart/app.py:188: if requires_sudo:
is this because of the new juju requirement?

https://codereview.appspot.com/174790043/

« Back to merge proposal