mirror of
https://github.com/ansible-collections/community.general.git
synced 2025-07-24 13:50:22 -07:00
Actually use changed variable
This commit is contained in:
parent
4125b576e9
commit
8ff71a3fa8
1 changed files with 5 additions and 6 deletions
|
@ -115,8 +115,8 @@ def main():
|
||||||
if module.check_mode:
|
if module.check_mode:
|
||||||
module.exit_json(changed=_system_state_change(module, secgroup))
|
module.exit_json(changed=_system_state_change(module, secgroup))
|
||||||
|
|
||||||
|
changed = False
|
||||||
if state == 'present':
|
if state == 'present':
|
||||||
changed = False
|
|
||||||
if not secgroup:
|
if not secgroup:
|
||||||
secgroup = cloud.create_security_group(name, description)
|
secgroup = cloud.create_security_group(name, description)
|
||||||
changed = True
|
changed = True
|
||||||
|
@ -126,14 +126,13 @@ def main():
|
||||||
secgroup['id'], description=description)
|
secgroup['id'], description=description)
|
||||||
changed = True
|
changed = True
|
||||||
module.exit_json(
|
module.exit_json(
|
||||||
changed=True, id=secgroup.id, secgroup=secgroup)
|
changed=changed, id=secgroup.id, secgroup=secgroup)
|
||||||
|
|
||||||
if state == 'absent':
|
if state == 'absent':
|
||||||
if not secgroup:
|
if secgroup:
|
||||||
module.exit_json(changed=False)
|
|
||||||
else:
|
|
||||||
cloud.delete_security_group(secgroup['id'])
|
cloud.delete_security_group(secgroup['id'])
|
||||||
module.exit_json(changed=True)
|
changed=True
|
||||||
|
module.exit_json(changed=changed)
|
||||||
|
|
||||||
except shade.OpenStackCloudException as e:
|
except shade.OpenStackCloudException as e:
|
||||||
module.fail_json(msg=e.message)
|
module.fail_json(msg=e.message)
|
||||||
|
|
Loading…
Add table
Add a link
Reference in a new issue