Skip to content

Commit 49a57f6

Browse files
authored
Merge pull request #90 from Yaish25491/ACA-1639_manage_security_group
ACA-1639 manage security group
2 parents 31a8585 + c928488 commit 49a57f6

File tree

2 files changed

+9
-6
lines changed

2 files changed

+9
-6
lines changed
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,2 @@
1+
bugfixes:
2+
- Adding alwayes statment to the first block in test_azure_manage_security_group main to ensure security group deleted

tests/integration/targets/test_azure_manage_security_group/tasks/main.yml

+7-6
Original file line numberDiff line numberDiff line change
@@ -11,6 +11,13 @@
1111
ansible.builtin.include_tasks: "create_and_delete_sg.yml"
1212
vars:
1313
test_resource_group: "{{ resource_group }}"
14+
15+
always:
16+
- name: Cleanup - make sure Security Group deleted - {{ security_group_name }}
17+
azure.azcollection.azure_rm_securitygroup:
18+
resource_group: "{{ resource_group }}"
19+
name: "{{ security_group_name }}"
20+
state: absent
1421

1522
- name: Test with create and delete Resource Group
1623
block:
@@ -32,12 +39,6 @@
3239
azure_manage_security_group_delete_resource_group: true
3340

3441
always:
35-
- name: Cleanup - make sure Security Group deleted - {{ security_group_name }}
36-
azure.azcollection.azure_rm_securitygroup:
37-
resource_group: "{{ resource_group }}"
38-
name: "{{ security_group_name }}"
39-
state: absent
40-
4142
- name: Cleanup - make sure Resource Group deleted - {{ new_resource_group }}
4243
azure.azcollection.azure_rm_resourcegroup:
4344
name: "{{ new_resource_group }}"

0 commit comments

Comments
 (0)