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

[Kaitlin Rupert]

>> @@ -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.

-- 
Kaitlin Rupert
IBM Linux Technology Center
kaitlin at linux.vnet.ibm.com