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
Please take a look.
https:/ /codereview. appspot. com/6850044/ diff/7017/ juju/providers/ ec2/securitygro up.py ec2/securitygro up.py (right):
File juju/providers/
https:/ /codereview. appspot. com/6850044/ diff/7017/ juju/providers/ ec2/securitygro up.py#newcode11 ec2/securitygro up.py:11: for group in d.result:
juju/providers/
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/