Code review comment for lp:~bac/charms/precise/juju-gui/increment-deployments

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

Just one more comment, I'll QA asap.

https://codereview.appspot.com/26740043/diff/20001/server/guiserver/bundles/base.py
File server/guiserver/bundles/base.py (right):

https://codereview.appspot.com/26740043/diff/20001/server/guiserver/bundles/base.py#newcode188
server/guiserver/bundles/base.py:188: if bundle_id is not None and error
is None:
I the future has been cancelled then error is not defined here.

https://codereview.appspot.com/26740043/

« Back to merge proposal