Re: [ovs-dev] [PATCH ovn v7 2/4] northd: Refactor Logical Flows for routers with DNAT/Load Balancers

2021-07-12 Thread Mark Gray
On 12/07/2021 05:53, Han Zhou wrote: > That's weird. These two tests just pass when I revert this commit. With > this commit it fails at: > > ../../tests/system-ovn.at:5752: ovs-appctl dpctl/dump-flows | grep > 'ct_state(-new+est+rpl+trk).*ct_label(0x.*04010204/.*)' -c > --- - 2021-07-

Re: [ovs-dev] [PATCH ovn v7 2/4] northd: Refactor Logical Flows for routers with DNAT/Load Balancers

2021-07-11 Thread Han Zhou
On Sun, Jul 11, 2021 at 11:49 AM Mark Gray wrote: > > > > > > Unfortunately there are constant system test failures: > > 141: ECMP IPv6 symmetric reply -- ovn-northd -- dp-groups=yes FAILED ( > > system-ovn.at:5752) > > 142: ECMP IPv6 symmetric reply -- ovn-northd -- dp-groups=no FAILED ( > > syst

Re: [ovs-dev] [PATCH ovn v7 2/4] northd: Refactor Logical Flows for routers with DNAT/Load Balancers

2021-07-11 Thread Mark Gray
> > Unfortunately there are constant system test failures: > 141: ECMP IPv6 symmetric reply -- ovn-northd -- dp-groups=yes FAILED ( > system-ovn.at:5752) > 142: ECMP IPv6 symmetric reply -- ovn-northd -- dp-groups=no FAILED ( > system-ovn.at:5752) > > Sorry that I didn't verify the system test

Re: [ovs-dev] [PATCH ovn v7 2/4] northd: Refactor Logical Flows for routers with DNAT/Load Balancers

2021-07-09 Thread Han Zhou
On Fri, Jul 9, 2021 at 2:02 PM Numan Siddique wrote: > > On Fri, Jul 9, 2021 at 4:17 PM Han Zhou wrote: > > > > On Wed, Jul 7, 2021 at 6:23 PM Han Zhou wrote: > > > > > > > > > > > > On Wed, Jul 7, 2021 at 1:28 AM Mark Gray wrote: > > > > > > > > This patch addresses a number of interconnected

Re: [ovs-dev] [PATCH ovn v7 2/4] northd: Refactor Logical Flows for routers with DNAT/Load Balancers

2021-07-09 Thread Numan Siddique
On Fri, Jul 9, 2021 at 4:17 PM Han Zhou wrote: > > On Wed, Jul 7, 2021 at 6:23 PM Han Zhou wrote: > > > > > > > > On Wed, Jul 7, 2021 at 1:28 AM Mark Gray wrote: > > > > > > This patch addresses a number of interconnected issues with Gateway > Routers > > > that have Load Balancing enabled: > >

Re: [ovs-dev] [PATCH ovn v7 2/4] northd: Refactor Logical Flows for routers with DNAT/Load Balancers

2021-07-09 Thread Han Zhou
On Wed, Jul 7, 2021 at 6:23 PM Han Zhou wrote: > > > > On Wed, Jul 7, 2021 at 1:28 AM Mark Gray wrote: > > > > This patch addresses a number of interconnected issues with Gateway Routers > > that have Load Balancing enabled: > > > > 1) In the router pipeline, we have the following stages to handl

Re: [ovs-dev] [PATCH ovn v7 2/4] northd: Refactor Logical Flows for routers with DNAT/Load Balancers

2021-07-07 Thread Han Zhou
On Wed, Jul 7, 2021 at 1:28 AM Mark Gray wrote: > > This patch addresses a number of interconnected issues with Gateway Routers > that have Load Balancing enabled: > > 1) In the router pipeline, we have the following stages to handle > dnat and unsnat. > > - Stage 4 : lr_in_defrag (dnat zone) >

Re: [ovs-dev] [PATCH ovn v7 2/4] northd: Refactor Logical Flows for routers with DNAT/Load Balancers

2021-07-07 Thread Mark Gray
On 07/07/2021 09:28, Mark Gray wrote: > This patch addresses a number of interconnected issues with Gateway Routers > that have Load Balancing enabled: > > 1) In the router pipeline, we have the following stages to handle > dnat and unsnat. > > - Stage 4 : lr_in_defrag (dnat zone) > - Stage 5 :

[ovs-dev] [PATCH ovn v7 2/4] northd: Refactor Logical Flows for routers with DNAT/Load Balancers

2021-07-07 Thread Mark Gray
This patch addresses a number of interconnected issues with Gateway Routers that have Load Balancing enabled: 1) In the router pipeline, we have the following stages to handle dnat and unsnat. - Stage 4 : lr_in_defrag (dnat zone) - Stage 5 : lr_in_unsnat (snat zone) - Stage 6 : lr_in_dnat (d