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

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

Reviewers: mp+237910_code.launchpad.net,

Message:
Please take a look.

Description:
Support automatic detection of a logged in MAAS.

Automatically detect a logged in MAAS API, so that
it is possible to use the stored credentials to
create and bootstrap a MAAS environment without
user intervention.

QA:
- ssh to the GUI MAAS;
- destroy the existing environment;
- remove the ~/.juju directory;
- use the MAAS UI (http://maas.jujugui.org/MAAS/nodes/)
   to release the nodes if they are not in a ready state;
- this branch is already checked out in ~/juju-quickstart/sandbox/;
- start quickstart from there:
   cd ~/juju-quickstart/sandbox/ && .venv/bin/python juju-quickstart
- quickstart should show the option to automatically create and
   bootstrap the MAAS environment;
- select the option and wait for the envirnment to be ready:
   this can fail due to juju/maas interaction problems we currently
   have, retrying the process should eventually succeed.

Done, thank you!

https://code.launchpad.net/~frankban/juju-quickstart/maas-detect/+merge/237910

(do not edit description out of merge proposal)

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

Affected files (+347, -12 lines):
   A [revision details]
   M quickstart/cli/views.py
   A quickstart/maas.py
   M quickstart/models/envs.py
   M quickstart/settings.py
   M quickstart/tests/cli/test_views.py
   M quickstart/tests/models/test_envs.py
   A quickstart/tests/test_maas.py

« Back to merge proposal