[Libvirt-cim] [PATCH] [TEST] Adding new tc to cover offline migration

Kaitlin Rupert kaitlin at linux.vnet.ibm.com
Mon Apr 6 17:58:00 UTC 2009


Deepti B Kalakeri wrote:
> 
> 
> Kaitlin Rupert wrote:
>>> +
>>> + at do_main(sup_types)
>>> +def main():
>>> +    options = main.options
>>> +    virt = options.virt
>>> +    s_sysname = gethostbyaddr(options.ip)[0]
>>> +    t_sysname = gethostbyaddr(options.t_url)[0] +    if options.virt 
>>> == 'KVM' and (t_sysname == s_sysname or \
>>> +       t_sysname in s_sysname):
>>> +        logger.info("Libvirt does not support local migratoin for KVM")
>>> +        return SKIP
>>
>> This should be a function.. since you'll need to cal it in the live 
>> migration case as well.
>>
>> Since this patch looks good otherwise, I've applied this as is.  Can 
>> you follow up with a patch to break this bit out into a function?
>>
> Yup I will do that.
> Also, I was not able to track the for which revision the actual changes 
> to the KVM migration was included as the RSS feed was not working today.
> Can you lemme know the same.
> 

It's revision 809 and changeset 6302880bc7b2.

The RSS feed is working for me, as is browsing the repo at: 
http://libvirt.org/hg/libvirt-cim/

Is this still not working for you?
-- 
Kaitlin Rupert
IBM Linux Technology Center
kaitlin at linux.vnet.ibm.com




More information about the Libvirt-cim mailing list