Remove unneed waits in test cases

plugins/modules/azure_rm_mysqldatabase.py

@@ -92,7 +92,7 @@
 
 try:
     from ansible_collections.azure.azcollection.plugins.module_utils.azure_rm_common import AzureRMModuleBase
-    from azure.core.exceptions import ResourceNotFoundError
+    from azure.core.exceptions import ResourceNotFoundError, HttpResponseError
     from azure.core.polling import LROPoller
 except ImportError:
     # This is handled in azure_rm_common
@@ -282,6 +282,8 @@ def get_mysqldatabase(self):
             self.log("MySQL Database instance : {0} found".format(response.name))
         except ResourceNotFoundError as e:
             self.log('Did not find the MySQL Database instance.')
+        except HttpResponseError as e:
+            self.log("Get MySQL Database instance error. code: {0}, message: {1}".format(e.status_code, str(e.error)))
         if found is True:
             return response.as_dict()
 

plugins/modules/azure_rm_mysqldatabase_info.py

@@ -101,7 +101,7 @@
 
 try:
     from ansible_collections.azure.azcollection.plugins.module_utils.azure_rm_common import AzureRMModuleBase
-    from azure.core.exceptions import ResourceNotFoundError
+    from azure.core.exceptions import ResourceNotFoundError, HttpResponseError
 except ImportError:
     # This is handled in azure_rm_common
     pass
@@ -159,6 +159,8 @@ def get(self):
             self.log("Response : {0}".format(response))
         except ResourceNotFoundError as e:
             self.log('Could not get facts for Databases.')
+        except HttpResponseError as e:
+            self.log("Get MySQL Database instance error. code: {0}, message: {1}".format(e.status_code, str(e.error)))
 
         if response is not None:
             results.append(self.format_item(response))

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

@@ -111,11 +111,6 @@
     name: "vmforimage{{ rpfx }}"
   register: output
 
-- name: Pause for 10 mimutes to VM updating
-  ansible.builtin.command:
-    sleep 600
-  changed_when: true
-
 - name: Create a snapshot by importing an unmanaged blob from the same subscription.
   azure_rm_snapshot:
     resource_group: "{{ resource_group }}"

tests/integration/targets/azure_rm_virtualmachine/tasks/azure_test_deallocate.yml

@@ -25,10 +25,6 @@
     name: "{{ vm_name }}"
   register: vm_state
 
-- name: Pause for 10 mimutes to VM updating
-  ansible.builtin.command: sleep 600
-  changed_when: true
-
 - name: Restart the virtual machine
   azure_rm_virtualmachine:
     resource_group: "{{ resource_group }}"

tests/integration/targets/azure_rm_virtualmachine/tasks/azure_test_dual_nic.yml

@@ -118,10 +118,6 @@
     name: "{{ vm_name }}"
   register: vm_state
 
-- name: Pause for 10 mimutes to VM updating
-  ansible.builtin.command: sleep 600
-  changed_when: true
-
 - name: Generalize VM
   azure_rm_virtualmachine:
     resource_group: "{{ resource_group }}"  # Should this be resource_group_secondary?

tests/integration/targets/azure_rm_virtualmachine/tasks/azure_test_ephemeral_os.yml

@@ -32,10 +32,6 @@
     name: "{{ vm_name }}"
   register: vm_state
 
-- name: Pause for 10 mimutes to VM updating
-  ansible.builtin.command: sleep 600
-  changed_when: true
-
 - name: Generalize VM
   azure_rm_virtualmachine:
     resource_group: "{{ resource_group }}"

tests/integration/targets/azure_rm_virtualmachine/tasks/azure_test_public_ip.yml

@@ -63,10 +63,6 @@
       #!/bin/sh
       echo "custom_data was executed" > /tmp/custom_data.txt
 
-- name: Pause for 10 mimutes to VM updating
-  ansible.builtin.command: sleep 600
-  changed_when: true
-
 - name: Ensure VM was created properly
   ansible.builtin.assert:
     that:

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

@@ -82,9 +82,6 @@
     resource_group: "{{ resource_group }}"
     name: "vmforimage{{ rpfx }}"
   register: vm_state
-- name: Pause for 10 mimutes to VM updating
-  ansible.builtin.command: sleep 600
-  changed_when: true
 - name: Generalize VM
   azure_rm_virtualmachine:
     resource_group: "{{ resource_group }}"

tests/integration/targets/inventory_azure/playbooks/setup.yml

@@ -47,7 +47,3 @@
           sku: 20_04-lts
           version: latest
       register: vm_output
-
-    - name: Pause for 5 mimutes for updating
-      ansible.builtin.command: sleep 300
-      changed_when: false