Do not poll to check if QEMU is alive

This allows to react quickly when QEMU finishes.

Signed-off-by: Akihiko Odaki <[email protected]>

lib/engines/hckinstall/hckinstall.rb

@@ -219,10 +219,8 @@ def run_studio_installer
                           @setup_studio_iso,
                           @studio_iso_info['path']
                         ], keep_alive: false, snapshot: false)
-        Timeout.timeout(@studio_install_timeout) do
-          @logger.info('Waiting for studio installation finished')
-          sleep 5 while st.alive?
-        end
+        @logger.info('Waiting for studio installation finished')
+        raise AutoHCKError, 'studio installation timed out' if st.wait(@studio_install_timeout).nil?
       end
     end
 
@@ -239,7 +237,7 @@ def run_clients_installer
         Timeout.timeout(@client_install_timeout) do
           cl.each do |name, client|
             @logger.info("Waiting for #{name} installation finished")
-            sleep 5 while client.alive?
+            client.wait
           end
         end
       end

lib/setupmanagers/physhck/physhck.rb

@@ -13,7 +13,7 @@ def initialize(logger)
         @logger = logger
       end
 
-      def alive?
+      def wait(...)
         @logger.info('Physical machine is always alive')
       end
 

lib/setupmanagers/qemuhck/qemu_machine.rb

@@ -106,8 +106,8 @@ def run_vm
         end
       end
 
-      def alive?
-        @qemu_thread.alive?
+      def wait(...)
+        @qemu_thread.join(...)
       end
 
       def soft_abort
@@ -133,7 +133,7 @@ def hard_abort
       def vm_abort
         @keep_alive = false
 
-        return unless alive?
+        return unless @qemu_thread.alive?
 
         return if soft_abort