[Compass-ci] Re: [PATCH v3 compass-ci] providers/multi-qemu: optimize

15 Mar 2021


      ...
-def reboot_deadline_machine(hostname)
-  mq = MQClient.new
+def loop_reboot_vm(hostname)
+  loop do
+    begin
+      reboot_vm(hostname)
+    rescue StandardError => e
+      puts e.backtrace
+      sleep 5
+    end
it will do the next with no sleep for the loop?
how about put 'sleep 5' follow the begin...end

Thanks, 
Luan Shengde
...
+  end
+end
+
+def reboot_vm(hostname)
+  mq = MQClient.new(MQ_HOST, MQ_PORT)
  queue = mq.queue(hostname, {:durable => true})
  queue.subscribe({:block => true, :manual_ack => true}) do |info,  _pro, msg|
    puts msg
@@ -118,10 +129,7 @@ end
def multiqemu
  Process.fork do
-    reboot_deadline_machine(HOSTNAME)
-  rescue StandardError => e
-    puts e.backtrace
-    sleep 5
+    loop_reboot_vm(HOSTNAME)
  end
pids = []
-- 
2.23.0

    