[Libvirt-cim] [PATCH 4 of 4] [TEST] #2 VSMS 08 Remove cleanup_guest_netpool() def and call it from vsmigration.py

[Deepti B Kalakeri]

Kaitlin Rupert wrote:
>>> @@ -135,6 +139,10 @@
>>>                  logger.error("Error setting up the guest")
>>>                  return status
>>>
>>> +            # Generally, having a test sleep is a bad choice, but 
>>> we need to
>>> +            # give the guest some time to fully boot before we 
>>> reboot it
>>> +            sleep(15)
>>>   
>> You can increase the timeout period in poll_for_state_change().
>> Did increasing the timeout period for poll_for_state_change() did not 
>> work for you ?
>
>
> The issue here is a slightly different.. in poll_for_state_change(), 
> we poll until libvirt reports that the guest is active.  However, 
> libvirt reports that the guest is active once the guest starts the 
> boot process.  So, pool_for_state_change() will stop polling, but the 
> guest may not have finished the boot sequence.
>
> At that point, we attempt to launch a restart migration - this means 
> that we attempt initiate a guest shutdown before the guest is even 
> booted.  For a guest shutdown, libvirt sends an ACPI event to the 
> guest, but if the guest isn't fully booted, it won't be able to detect 
> that event.
>
> So we need to give the OS a chance to boot before asking for a shutdown.
Oh! this makes sense then...

-- 
Thanks and Regards,
Deepti B. Kalakeri
IBM Linux Technology Center
deeptik at linux.vnet.ibm.com