JSON support + minor fixes.

JSON support + minor fixes.

This branch includes official JSON
support for bundle deployments.

This branch also contains a number of
minor fixes/improvements:

- Added the possibility to provide a
  customized Juju GUI charm URL.

- Added support for providing a bundle 
  directory (a dir containing a 
  bundles.yaml file): fix bug 1247181.

- Some documentation clean up.

- Added an option to decline opening the 
  browser.

- The Juju env is bootstrapped passing 
  --debug when quickstart is run in debug
  mode.

- Bumped version up.

Tests: `make check`.

QA:
I uploaded a bundle JSON here:
http://pastebin.com/raw.php?i=4mDCcagp
So it is possible to check the new features 
by running the following (expect a scary output):
.venv/bin/python juju-quickstart --debug --no-browser \
	--gui-charm-url cs:~juju-gui/precise/juju-gui-116 \
	-e ec2 http://pastebin.com/raw.php?i=4mDCcagp
At the end of the process, locate the GUI url
printed before the last debug messages, and check
the GUI is up, running and the bundle is deploying. 
Remember to destroy your ec2 env.
Thank you!

https://code.launchpad.net/~frankban/juju-quickstart/support-json/+merge/194172

(do not edit description out of merge proposal)

[frankban, 2013-11-06 15:43:51]

Please take a look.

[matthew.scott, 2013-11-06 17:23:16]

LGTM, QA Okay.  Thanks!

[gary.poster, 2013-11-06 17:40:43]

LGTM with trivial.  Thank you!

https://codereview.appspot.com/22300044/diff/1/quickstart/app.py
File quickstart/app.py (right):

https://codereview.appspot.com/22300044/diff/1/quickstart/app.py#newcode123
quickstart/app.py:123: and the optional Juju GUI charm URL. If the charm URL is
not provided, the
Maybe give an example of the expected format, e.g.
cs:~juju-gui/precise/juju-gui-116 ?  Maybe that's more valuable/important for
the commandline help.

https://codereview.appspot.com/22300044/diff/1/quickstart/manage.py
File quickstart/manage.py (right):

https://codereview.appspot.com/22300044/diff/1/quickstart/manage.py#newcode181
quickstart/manage.py:181: 'provided, the last release of the GUI will be
deployed')
This would be the ideal place to include an example IMO, like mentioning that
you can get the dev charm by using cs:~juju-gui/precise/juju-gui

[frankban, 2013-11-06 18:01:12]

*** Submitted:

JSON support + minor fixes.

This branch includes official JSON
support for bundle deployments.

This branch also contains a number of
minor fixes/improvements:

- Added the possibility to provide a
  customized Juju GUI charm URL.

- Added support for providing a bundle 
  directory (a dir containing a 
  bundles.yaml file): fix bug 1247181.

- Some documentation clean up.

- Added an option to decline opening the 
  browser.

- The Juju env is bootstrapped passing 
  --debug when quickstart is run in debug
  mode.

- Bumped version up.

Tests: `make check`.

QA:
I uploaded a bundle JSON here:
http://pastebin.com/raw.php?i=4mDCcagp
So it is possible to check the new features 
by running the following (expect a scary output):
.venv/bin/python juju-quickstart --debug --no-browser \
	--gui-charm-url cs:~juju-gui/precise/juju-gui-116 \
	-e ec2 http://pastebin.com/raw.php?i=4mDCcagp
At the end of the process, locate the GUI url
printed before the last debug messages, and check
the GUI is up, running and the bundle is deploying. 
Remember to destroy your ec2 env.
Thank you!

R=
CC=
https://codereview.appspot.com/22300044

https://codereview.appspot.com/22300044/diff/1/quickstart/app.py
File quickstart/app.py (right):

https://codereview.appspot.com/22300044/diff/1/quickstart/app.py#newcode123
quickstart/app.py:123: and the optional Juju GUI charm URL. If the charm URL is
not provided, the
On 2013/11/06 17:40:43, gary.poster wrote:
> Maybe give an example of the expected format, e.g.
> cs:~juju-gui/precise/juju-gui-116 ?  Maybe that's more valuable/important for
> the commandline help.

Done.

https://codereview.appspot.com/22300044/diff/1/quickstart/manage.py
File quickstart/manage.py (right):

https://codereview.appspot.com/22300044/diff/1/quickstart/manage.py#newcode181
quickstart/manage.py:181: 'provided, the last release of the GUI will be
deployed')
On 2013/11/06 17:40:43, gary.poster wrote:
> This would be the ideal place to include an example IMO, like mentioning that
> you can get the dev charm by using cs:~juju-gui/precise/juju-gui

Done.

[frankban, 2013-11-06 18:02:10]

Thank you both!