Adds variables to control reboot (#12)

Based on feedback in: https://review.opendev.org/c/openstack/kayobe/+/887200
stackhpc · Aug 2, 2023 · aa8b5bf · aa8b5bf
1 parent 17d1349
commit aa8b5bf
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/roles/vgpu/defaults/main.yml b/roles/vgpu/defaults/main.yml
@@ -2,6 +2,8 @@
 vgpu_driver_url: ""
 vgpu_driver_force_install: false
 vgpu_driver_dkms: false
+vgpu_do_reboot: true
+vgpu_reboot_timeout: 3600
 
 # Deprecated: use vgpu_definitions instead.
 vgpu_mig_definitions: []

diff --git a/roles/vgpu/tasks/install.yml b/roles/vgpu/tasks/install.yml
@@ -65,9 +65,11 @@
 
     - name: Reboot after driver install # noqa: no-handler
       ansible.builtin.reboot:
-        reboot_timeout: 3600
+        reboot_timeout: "{{ vgpu_reboot_timeout }}"
         post_reboot_delay: 60
-      when: install_result is changed
+      when:
+        - install_result is changed
+        - vgpu_do_reboot | bool
 
 - name: Install nvidia-mig-manager (Redhat)
   ansible.builtin.package: