Re: [ovs-dev] [PATCH 7/7] ovn-northd: Logical flows for load balancers.

2016-07-05 Thread Guru Shetty
On 5 July 2016 at 07:31, Zong Kai LI wrote: > Hi, Ben and Guru. I tried to test lb feature on my OpenStack env, but > failed. > The simplest topology, three VMs(cirros) and VIP are on the same switch. > VM2 and VM3 are endpoints for the VIP. > I tried to use ping and ssh to test VIP, but things d

Re: [ovs-dev] [PATCH 7/7] ovn-northd: Logical flows for load balancers.

2016-07-05 Thread Guru Shetty
On 5 July 2016 at 07:31, Zong Kai LI wrote: > Hi, Ben and Guru. I tried to test lb feature on my OpenStack env, but > failed. > The simplest topology, three VMs(cirros) and VIP are on the same switch. > VM2 and VM3 are endpoints for the VIP. > I tried to use ping and ssh to test VIP, but things d

Re: [ovs-dev] [PATCH 7/7] ovn-northd: Logical flows for load balancers.

2016-07-05 Thread Zong Kai LI
Hi, Ben and Guru. I tried to test lb feature on my OpenStack env, but failed. The simplest topology, three VMs(cirros) and VIP are on the same switch. VM2 and VM3 are endpoints for the VIP. I tried to use ping and ssh to test VIP, but things don't work. I think it should be arp issue. First, in ta

Re: [ovs-dev] [PATCH 7/7] ovn-northd: Logical flows for load balancers.

2016-07-03 Thread Ben Pfaff
On Sun, Jul 03, 2016 at 08:17:32PM -0700, Guru Shetty wrote: > On 3 July 2016 at 10:24, Ben Pfaff wrote: > > > On Wed, Jun 29, 2016 at 01:17:11AM -0700, Gurucharan Shetty wrote: > > > This commit adds a 'pre_lb' table that sits before 'pre_stateful' table. > > > For packets that need to be load b

Re: [ovs-dev] [PATCH 7/7] ovn-northd: Logical flows for load balancers.

2016-07-03 Thread Guru Shetty
On 3 July 2016 at 10:24, Ben Pfaff wrote: > On Wed, Jun 29, 2016 at 01:17:11AM -0700, Gurucharan Shetty wrote: > > This commit adds a 'pre_lb' table that sits before 'pre_stateful' table. > > For packets that need to be load balanced, this table sets reg0[0] > > to act as a hint for the pre-state

Re: [ovs-dev] [PATCH 7/7] ovn-northd: Logical flows for load balancers.

2016-07-03 Thread Ben Pfaff
On Wed, Jun 29, 2016 at 01:17:11AM -0700, Gurucharan Shetty wrote: > This commit adds a 'pre_lb' table that sits before 'pre_stateful' table. > For packets that need to be load balanced, this table sets reg0[0] > to act as a hint for the pre-stateful table to send the packet to > the conntrack tabl

Re: [ovs-dev] [PATCH 7/7] ovn-northd: Logical flows for load balancers.

2016-07-01 Thread Guru Shetty
> > > So the only case to make pre_lb tables are necessary is, logical switch > doesn't contain an "allow-related" action ACL rule. It seems possible, but > I cannot figure out why people choose to not using "allow-related" action, > that will make ACL table hard to maintain. > That is a bad assum

Re: [ovs-dev] [PATCH 7/7] ovn-northd: Logical flows for load balancers.

2016-07-01 Thread Zong Kai LI
Hi, Guru. > +/* 'REGBIT_CONNTRACK_DEFRAG' is set to let the pre-stateful table > send > + * packet to conntrack for defragmentation. */ +const char *ip_address; > +SSET_FOR_EACH(ip_address, &all_ips) { > +char *match = xasprintf("ip && ip4.dst == %s",

[ovs-dev] [PATCH 7/7] ovn-northd: Logical flows for load balancers.

2016-06-29 Thread Gurucharan Shetty
This commit adds a 'pre_lb' table that sits before 'pre_stateful' table. For packets that need to be load balanced, this table sets reg0[0] to act as a hint for the pre-stateful table to send the packet to the conntrack table for defragmentation. This commit also adds a 'lb' table that sits before