Skip to content

Commit d22ca22

Browse files
committed
Remove comment
Signed-off-by: Webster Mudge <[email protected]>
1 parent 7aa18a4 commit d22ca22

File tree

2 files changed

+7
-3
lines changed

2 files changed

+7
-3
lines changed

roles/data/tasks/setup_azure.yml

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -27,12 +27,16 @@
2727
loop_var: __data_store_azure_location
2828

2929
- name: Process Azure role assignments for external data storage locations
30-
azure.azcollection.azure_rm_roleassignment: # This Azure module is not idempotent on removals
30+
azure.azcollection.azure_rm_roleassignment:
3131
state: present
3232
scope: "{{ __azure_data_location_assignment.scope }}"
3333
name: "{{ __azure_data_location_assignment.name }}"
3434
assignee_object_id: "{{ __azure_data_location_assignment.assignee }}"
3535
role_definition_id: "{{ __data_store.read_only | default(False) | ternary(__azure_storage_blob_data_reader_uri, __azure_storage_blob_data_owner_uri) }}"
3636
loop: "{{ __azure_data_location_storage_assignments }}"
3737
loop_control:
38-
loop_var: __azure_data_location_assignment
38+
loop_var: __azure_data_location_assignment
39+
register: __data_azure_assignment_results
40+
until: __data_azure_assignment_results is not failed
41+
retries: 3
42+
delay: 3

roles/data/tasks/teardown_azure.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@
2828
when: __data_storage.assignment.delete | default(data__teardown_deletes_assignments)
2929
block:
3030
- name: Process Azure role assignment removal for external data storage locations
31-
azure.azcollection.azure_rm_roleassignment: # This Azure module is not idempotent on removals
31+
azure.azcollection.azure_rm_roleassignment:
3232
state: absent
3333
scope: "{{ __azure_data_location_assignment.scope }}"
3434
assignee_object_id: "{{ __azure_data_location_assignment.assignee }}"

0 commit comments

Comments
 (0)