Re: [ovs-dev] [PATCH v9] ovn-nbctl: Add LB commands.

2016-09-30 Thread nickcooper-zhangtonghao
Thanks for your tips, I fixed the memory leak(via free(ips)). The patch follows the style of the original code. So the 'const' is not used with 'may_exist', 'add_duplicate' and 'must_exist'. You can find that in files, such as 'utilities/ovs-vsctl.c', 'ovn/utilities/ovn-sbctl.c' and 'ovn/utiliti

Re: [ovs-dev] [PATCH v9] ovn-nbctl: Add LB commands.

2016-09-29 Thread Flavio Fernandes
> On Sep 29, 2016, at 5:43 AM, nickcooper-zhangtonghao > wrote: > > This patch provides the command line to create a load balancer. > You can create a load balancer independently and add it to multiple > switches or routers. A single load balancer can have multiple vips. > Add a name column for

Re: [ovs-dev] [PATCH v9] ovn-nbctl: Add LB commands.

2016-09-29 Thread Guru Shetty
On 29 September 2016 at 02:43, nickcooper-zhangtonghao < nickcooper-zhangtong...@opencloud.tech> wrote: > This patch provides the command line to create a load balancer. > You can create a load balancer independently and add it to multiple > switches or routers. A single load balancer can have mul

[ovs-dev] [PATCH v9] ovn-nbctl: Add LB commands.

2016-09-29 Thread nickcooper-zhangtonghao
This patch provides the command line to create a load balancer. You can create a load balancer independently and add it to multiple switches or routers. A single load balancer can have multiple vips. Add a name column for the load balancer. With --add-duplicate, the command really creates a new loa