Remove comment

Signed-off-by: Webster Mudge <[email protected]>
cloudera-labs · Apr 1, 2022 · aa1d67f · aa1d67f
1 parent 08e4548
commit aa1d67f
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 3 deletions.
diff --git a/roles/data/tasks/setup_azure.yml b/roles/data/tasks/setup_azure.yml
@@ -27,12 +27,16 @@
     loop_var: __data_store_azure_location
 
 - name: Process Azure role assignments for external data storage locations
-  azure.azcollection.azure_rm_roleassignment: # This Azure module is not idempotent on removals
+  azure.azcollection.azure_rm_roleassignment:
     state: present
     scope: "{{ __azure_data_location_assignment.scope }}"
     name: "{{ __azure_data_location_assignment.name }}"
     assignee_object_id: "{{ __azure_data_location_assignment.assignee }}"
     role_definition_id: "{{ __data_store.read_only | default(False) | ternary(__azure_storage_blob_data_reader_uri, __azure_storage_blob_data_owner_uri) }}"
   loop: "{{ __azure_data_location_storage_assignments }}"
   loop_control:
-    loop_var: __azure_data_location_assignment
+    loop_var: __azure_data_location_assignment
+  register: __data_azure_assignment_results
+  until: __data_azure_assignment_results is not failed
+  retries: 3
+  delay: 3
diff --git a/roles/data/tasks/teardown_azure.yml b/roles/data/tasks/teardown_azure.yml
@@ -28,7 +28,7 @@
   when: __data_storage.assignment.delete | default(data__teardown_deletes_assignments)
   block:
     - name: Process Azure role assignment removal for external data storage locations
-      azure.azcollection.azure_rm_roleassignment: # This Azure module is not idempotent on removals
+      azure.azcollection.azure_rm_roleassignment:
         state: absent
         scope: "{{ __azure_data_location_assignment.scope }}"
         assignee_object_id: "{{ __azure_data_location_assignment.assignee }}"