virt script timeout is implemented incorrectly

Bug #1628264 reported by Jeff Lane 
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Checkbox Provider - Base
Fix Released
Medium
Jeff Lane 

Bug Description

the current implementation that boots the VM is this:

 # Boot Virtual Machine
 instance = self.boot_image(self.image)

 time.sleep(self.timeout)
 # If running in console, reset console window to regain
 # control from VM Serial I/0

This, unfortunately, makes the test run a lot longer than necessary.

Instead, we should run it so that the boot is kicked off and then we poll the output file every X seconds until timeout is reached.

That way, if we get the CERTIFICATION BOOT COMPLETE message before the timeout, we can exit the loop and proceed without waiting the full timeout time, OR we just time out, check one last time for the message and error if we don't see it.

Jeff Lane  (bladernr)
Changed in plainbox-provider-checkbox:
status: Triaged → In Progress
assignee: nobody → Jeff Lane (bladernr)
milestone: none → future
Jeff Lane  (bladernr)
Changed in plainbox-provider-checkbox:
status: In Progress → Fix Committed
Changed in plainbox-provider-checkbox:
milestone: future → 0.37.0
Changed in plainbox-provider-checkbox:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.