Code review comment for lp:~franciscosouza/pyjuju/juju-vpc

Revision history for this message
Chico (franciscossouza) wrote :

Please take a look.

https://codereview.appspot.com/6850044/diff/7017/juju/providers/ec2/securitygroup.py
File juju/providers/ec2/securitygroup.py (right):

https://codereview.appspot.com/6850044/diff/7017/juju/providers/ec2/securitygroup.py#newcode11
juju/providers/ec2/securitygroup.py:11: for group in d.result:
On 2012/11/22 13:44:07, hazmat wrote:
> This is very suspect, deferred results should not be access directly,
this needs
> a callback registered or inlineCallbacks syntax.

Done.

https://codereview.appspot.com/6850044/

« Back to merge proposal