From 4125b576e9ebe89d9977ed4c9cf88d90d0ee1135 Mon Sep 17 00:00:00 2001 From: Monty Taylor Date: Wed, 17 Jun 2015 08:02:34 -0400 Subject: [PATCH] Return the secgroup for all present states --- .../modules/cloud/openstack/os_security_group.py | 11 +++++------ 1 file changed, 5 insertions(+), 6 deletions(-) diff --git a/lib/ansible/modules/cloud/openstack/os_security_group.py b/lib/ansible/modules/cloud/openstack/os_security_group.py index 4f929e62f4b..48aed418fa4 100644 --- a/lib/ansible/modules/cloud/openstack/os_security_group.py +++ b/lib/ansible/modules/cloud/openstack/os_security_group.py @@ -116,18 +116,17 @@ def main(): module.exit_json(changed=_system_state_change(module, secgroup)) if state == 'present': + changed = False if not secgroup: secgroup = cloud.create_security_group(name, description) - module.exit_json( - changed=True, id=secgroup.id, secgroup=secgroup) + 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, secgroup=secgroup) - else: - module.exit_json(changed=False) + changed = True + module.exit_json( + changed=True, id=secgroup.id, secgroup=secgroup) if state == 'absent': if not secgroup: