feat(bmc): add bmc_reboot_should_wait argument

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

docs/bmc_reboot_role.rst

@@ -108,6 +108,92 @@ Parameters
       Security token for authentication with DPU BMC.
 
 
+      .. raw:: html
+
+        </div>
+
+  * - .. raw:: html
+
+        <div class="ansible-option-cell">
+        <div class="ansibleOptionAnchor" id="parameter-main--bmc_reboot_should_wait"></div>
+
+      .. _ansible_collections.opiproject.dpu.bmc_reboot_role__parameter-main__bmc_reboot_should_wait:
+
+      .. rst-class:: ansible-option-title
+
+      **bmc_reboot_should_wait**
+
+      .. raw:: html
+
+        <a class="ansibleOptionLink" href="#parameter-main--bmc_reboot_should_wait" title="Permalink to this option"></a>
+
+      .. ansible-option-type-line::
+
+        :ansible-option-type:`boolean`
+
+
+
+
+      .. raw:: html
+
+        </div>
+
+    - .. raw:: html
+
+        <div class="ansible-option-cell">
+
+      Wait for the BMC to restart.
+
+
+      .. rst-class:: ansible-option-line
+
+      :ansible-option-choices:`Choices:`
+
+      - :ansible-option-choices-entry:`false`
+      - :ansible-option-choices-entry-default:`true` :ansible-option-choices-default-mark:`← (default)`
+
+
+      .. raw:: html
+
+        </div>
+
+  * - .. raw:: html
+
+        <div class="ansible-option-cell">
+        <div class="ansibleOptionAnchor" id="parameter-main--bmc_reboot_wait_timeout"></div>
+
+      .. _ansible_collections.opiproject.dpu.bmc_reboot_role__parameter-main__bmc_reboot_wait_timeout:
+
+      .. rst-class:: ansible-option-title
+
+      **bmc_reboot_wait_timeout**
+
+      .. raw:: html
+
+        <a class="ansibleOptionLink" href="#parameter-main--bmc_reboot_wait_timeout" title="Permalink to this option"></a>
+
+      .. ansible-option-type-line::
+
+        :ansible-option-type:`integer`
+
+
+
+
+      .. raw:: html
+
+        </div>
+
+    - .. raw:: html
+
+        <div class="ansible-option-cell">
+
+      Seconds to wait for the BMC to restart.
+
+
+      .. rst-class:: ansible-option-line
+
+      :ansible-option-default-bold:`Default:` :ansible-option-default:`120`
+
       .. raw:: html
 
         </div>

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.6.0",
+  "community.general": ">=9.1.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: 120

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: 120

roles/bmc_reboot/tasks/main.yml

@@ -28,10 +28,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
@@ -44,7 +42,5 @@
         timeout: 60
         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 }}"