feat(bmc): add bmc_reboot_should_wait argument

Signed-off-by: Boris Glimcher <[email protected]>

galaxy.yml

@@ -42,7 +42,7 @@ tags: [opi, dpu, redfish, automation, devops, infrastructure]
 # L(specifiers,https://python-semanticversion.readthedocs.io/en/latest/#requirement-specification). Multiple version
 # range specifiers can be set and are separated by ','
 dependencies: {
-  "community.general": ">=8.4.0",
+  "community.general": ">=8.6.0",
 }
 
 # The URL of the originating SCM repository

roles/bmc_reboot/defaults/main.yml

@@ -5,3 +5,5 @@
 # defaults file for bmc_reboot
 
 https_port: 443
+bmc_reboot_should_wait: true
+bmc_reboot_wait_timeout: 60

roles/bmc_reboot/meta/argument_specs.yml

@@ -29,3 +29,13 @@ argument_specs:
         description:
           - DPU BMC port.
         default: 443
+      bmc_reboot_should_wait:
+        description:
+          - Wait for the BMC to restart.
+        type: bool
+        default: true
+      bmc_reboot_wait_timeout:
+        description:
+          - Seconds to wait for the BMC to restart.
+        type: int
+        default: 60

roles/bmc_reboot/tasks/main.yml

@@ -27,10 +27,8 @@
         baseuri: "{{ inventory_hostname }}"
         username: "{{ dpu_bmc_username }}"
         password: "{{ dpu_bmc_password }}"
-
-    - name: Pause for 60 seconds for BMC to Reboot
-      ansible.builtin.pause:
-        seconds: 60
+        wait: "{{ bmc_reboot_should_wait }}"
+        wait_timeout: "{{ bmc_reboot_wait_timeout }}"
 
 - name: Reboot BMC when auth_token is defined
   when: dpu_bmc_token is defined
@@ -42,7 +40,5 @@
         command: GracefulRestart
         baseuri: "{{ inventory_hostname }}"
         auth_token: "{{ dpu_bmc_token }}"
-
-    - name: Pause for 60 seconds for BMC to Reboot
-      ansible.builtin.pause:
-        seconds: 60
+        wait: "{{ bmc_reboot_should_wait }}"
+        wait_timeout: "{{ bmc_reboot_wait_timeout }}"