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

[Han Zhou]

On Sun, Jul 11, 2021 at 11:49 AM Mark Gray <mark.d.gray at redhat.com> 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 (
> > system-ovn.at:5752)
> >
> > Sorry that I didn't verify the system test before merging. Mark/Numan,
> > would you take a look?
> >
>
> Thanks for merging, and the review, and Ariel's testing. These failures
> were here before the merge and are not caused by this patch. I have been
> working on a fix for them.
>
> The problem is IPv6 NS and RS flows are evicting flows causing the test
> to fail. I have it in a state that works much better but still
> occasionally fails.
>

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.*00000401020400000000/.*)' -c
--- - 2021-07-11 21:49:01.127154470 -0700
+++
/home/hanzhou/src/ovn-system-test/ovn/_build/tests/system-kmod-testsuite.dir/at-groups/141/stdout
2021-07-11 21:49:01.124053642 -0700
@@ -1,2 +1,2 @@
-1
+0

Seems the CT state is not committed in that scenario which causes the
expected DP flow missing. I didn't dig further yet.

Thanks,
Han