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

[Deepti B Kalakeri]

Kaitlin Rupert wrote:
> 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?
Yup this works now. Thanks

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