feat(bmc): add bmc_reboot_should_wait argument

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

roles/bmc_fw_update/tasks/main.yml

@@ -67,10 +67,6 @@
   when:
     - bmc_fw_update_reboot is true
 
-- name: Pause for 60 seconds for BMC to Reboot
-  ansible.builtin.pause:
-    seconds: 60
-
 - name: Get Firmware Inventory
   ansible.builtin.include_role:
     name: get_bmc_facts

roles/bmc_reboot/defaults/main.yml

@@ -1,2 +1,7 @@
+# SPDX-License-Identifier: Apache-2.0
+# Copyright (c) 2022-2024 Dell Inc, or its subsidiaries.
+
 ---
 # defaults file for bmc_reboot
+
+bmc_reboot_should_wait: true

roles/bmc_reboot/meta/argument_specs.yml

@@ -0,0 +1,15 @@
+# SPDX-License-Identifier: Apache-2.0
+# Copyright (c) 2022-2024 Dell Inc, or its subsidiaries.
+---
+argument_specs:
+  main:
+    version_added: "1.0.0"
+    short_description: Role to restart DPU BMC
+    description:
+      - Role to restart DPU BMC.
+    options:
+      bmc_reboot_should_wait:
+        description:
+          - Wait for the BMC to restart.
+        type: bool
+        default: true

roles/bmc_reboot/tasks/main.yml

@@ -12,3 +12,9 @@
     username: "{{ dpu_bmc_username }}"
     password: "{{ dpu_bmc_password }}"
   delegate_to: localhost
+
+- name: Pause for a bit to allow BMC to Reboot
+  ansible.builtin.pause:
+    seconds: "{{ bmc_reboot_delay }}"
+  when:
+    - bmc_reboot_should_wait is true

roles/bmc_reboot/vars/main.yml

@@ -1,2 +1,7 @@
+# SPDX-License-Identifier: Apache-2.0
+# Copyright (c) 2022-2024 Dell Inc, or its subsidiaries.
+
 ---
 # vars file for bmc_reboot
+
+bmc_reboot_delay: 60