Re: [libvirt] [PATCH] util: do not take it as wrong if no PortData is found while getting migrateData

2015-02-26 Thread Martin Kletzander
On Thu, Feb 26, 2015 at 10:25:45AM +0100, Martin Kletzander wrote: On Thu, Feb 12, 2015 at 12:08:54PM +0800, Zhang Bo wrote: The function virNetDevOpenvswitchGetMigrateData() uses the cmd ovs-vsctl to get portdata. If no portdata is available, rather than failure in running the cmd, we think we

Re: [libvirt] [PATCH] util: do not take it as wrong if no PortData is found while getting migrateData

2015-02-26 Thread Martin Kletzander
On Thu, Feb 12, 2015 at 12:08:54PM +0800, Zhang Bo wrote: The function virNetDevOpenvswitchGetMigrateData() uses the cmd ovs-vsctl to get portdata. If no portdata is available, rather than failure in running the cmd, we think we should just print a warning message here, rather than taking it as w

Re: [libvirt] [PATCH] util: do not take it as wrong if no PortData is found while getting migrateData

2015-02-25 Thread zhang bo
On 2015/2/20 18:05, Martin Kletzander wrote: > On Thu, Feb 12, 2015 at 12:08:54PM +0800, Zhang Bo wrote: >> The function virNetDevOpenvswitchGetMigrateData() uses the cmd ovs-vsctl to >> get portdata. If no portdata is available, rather than failure in running >> the cmd, we think we should just p

Re: [libvirt] [PATCH] util: do not take it as wrong if no PortData is found while getting migrateData

2015-02-20 Thread Martin Kletzander
On Thu, Feb 12, 2015 at 12:08:54PM +0800, Zhang Bo wrote: The function virNetDevOpenvswitchGetMigrateData() uses the cmd ovs-vsctl to get portdata. If no portdata is available, rather than failure in running the cmd, we think we should just print a warning message here, rather than taking it as w

[libvirt] [PATCH] util: do not take it as wrong if no PortData is found while getting migrateData

2015-02-11 Thread Zhang Bo
The function virNetDevOpenvswitchGetMigrateData() uses the cmd ovs-vsctl to get portdata. If no portdata is available, rather than failure in running the cmd, we think we should just print a warning message here, rather than taking it as wrong, because PortData is just optional for an ovs interface