Code review comment for lp:~frankban/charms/precise/juju-gui/cancel-deployment

Revision history for this message
Richard Harding (rharding) wrote :

LGTM

Results of the script, first run:

http://paste.mitechie.com/show/1020/

I did run it a second time and got an error:

http://paste.mitechie.com/show/1021/

I'm not sure if this is a bug here or as a follow up to not crash when
the service is already there.

https://codereview.appspot.com/13549046/

« Back to merge proposal