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

Deepti B Kalakeri wrote: > > > Kaitlin Rupert wrote: >>> + >>> +@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?