cmueller/ansible
1
0
Fork 0
Code Issues Pull requests Projects Releases Wiki Activity

delete subnet should be idempotent (#35037)

Browse source
This commit is contained in:
Yuwei Zhou 2018-01-25 08:59:14 +08:00 committed by Jordan Borean
parent e9e85625aa
commit 7c201fc9bc
2 changed files with 12 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
lib/ansible/modules/cloud/azure/azure_rm_subnet.py
View file

@ -254,7 +254,7 @@ class AzureRMSubnet(AzureRMModuleBase):
resource_guid=nsg.resource_guid) resource_guid=nsg.resource_guid)
self.results['state'] = self.create_or_update_subnet(subnet) self.results['state'] = self.create_or_update_subnet(subnet)
elif self.state == 'absent': elif self.state == 'absent' and changed:
# delete subnet # delete subnet
self.delete_subnet() self.delete_subnet()
# the delete does not actually return anything. if no exception, then we'll assume # the delete does not actually return anything. if no exception, then we'll assume

11
test/integration/targets/azure_rm_subnet/tasks/main.yml
View file

@ -82,6 +82,17 @@
virtual_network_name: My_Virtual_Network virtual_network_name: My_Virtual_Network
resource_group: "{{ resource_group }}" resource_group: "{{ resource_group }}"
- name: Remove subnet (idempotent)
azure_rm_subnet:
state: absent
name: foobar
virtual_network_name: My_Virtual_Network
resource_group: "{{ resource_group }}"
register: output
- assert:
that: not output.changed
- name: Remove security group - name: Remove security group
azure_rm_securitygroup: azure_rm_securitygroup:
resource_group: "{{ resource_group }}" resource_group: "{{ resource_group }}"