From 94a8b6dcccfcceb6cbec876ad957ed2c0a105c19 Mon Sep 17 00:00:00 2001
From: Monty Taylor <mordred@inaugust.com>
Date: Fri, 19 Jun 2015 17:06:12 -0400
Subject: [PATCH] Make sure we're always returning objects too
---
cloud/openstack/os_security_group.py | 16 ++++++++--------
1 file changed, 8 insertions(+), 8 deletions(-)
diff --git a/cloud/openstack/os_security_group.py b/cloud/openstack/os_security_group.py
index 86e6de0b023..7fba28c8cb9 100644
--- a/cloud/openstack/os_security_group.py
+++ b/cloud/openstack/os_security_group.py
@@ -116,24 +116,24 @@ def main():
if module.check_mode:
module.exit_json(changed=_system_state_change(module, secgroup))
+ changed = False
if state == 'present':
if not secgroup:
secgroup = cloud.create_security_group(name, description)
- module.exit_json(changed=True, id=secgroup['id'])
+ changed = True
else:
if _needs_update(module, secgroup):
secgroup = cloud.update_security_group(
secgroup['id'], description=description)
- module.exit_json(changed=True, id=secgroup['id'])
- else:
- module.exit_json(changed=False, id=secgroup['id'])
+ changed = True
+ module.exit_json(
+ changed=changed, id=secgroup['id'], secgroup=secgroup)
if state == 'absent':
- if not secgroup:
- module.exit_json(changed=False)
- else:
+ if secgroup:
cloud.delete_security_group(secgroup['id'])
- module.exit_json(changed=True)
+ changed = True
+ module.exit_json(changed=changed)
except shade.OpenStackCloudException as e:
module.fail_json(msg=e.message)