virt script timeout is implemented incorrectly
Bug #1628264 reported by
Jeff Lane
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_
time.sleep(
# 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.
Related branches
~bladernr/plainbox-provider-checkbox:1628264-virt-timeout
- Rod Smith: Approve
-
Diff: 167 lines (+45/-32)1 file modifiedbin/virtualization (+45/-32)
Changed in plainbox-provider-checkbox: | |
status: | Triaged → In Progress |
assignee: | nobody → Jeff Lane (bladernr) |
milestone: | none → future |
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.