[ovs-dev] [PATCH ovn] Revert "ovn-controller: Handle DNAT/no-NAT conntrack tuple collisions."

[Flavio Fernandes]

This revert does not include a 1 liner change needed done later in
https://github.com/ovn-org/ovn/commit/4e6c498068dc4fa9546d3661f78f0a42e99c74bb <https://github.com/ovn-org/ovn/commit/4e6c498068dc4fa9546d3661f78f0a42e99c74bb>
That is not an issue, since regex change there is backwards compatible, agree?


Acked-by: Flavio Fernandes <flavio at flaviof.com>


> On Aug 27, 2021, at 11:29 AM, Mark Michelson <mmichels at redhat.com> wrote:
> 
> This commit resulted in significant decreased dataplane performance when
> testing a dense OpenShift cluster. This was pinpointed to be due to an
> extra ct(nat(src)) that this commit added.
> 
> For now, revert this commit.
> 
> Reported-at: https://bugzilla.redhat.com/show_bug.cgi?id=1992012
> Signed-off-by: Mark Michelson <mmichels at redhat.com>
> ---
> include/ovn/actions.h         |   1 -
> lib/actions.c                 |  31 ------
> tests/ovn.at                  |   2 +-
> tests/system-common-macros.at |   4 -
> tests/system-ovn.at           | 190 ----------------------------------
> 5 files changed, 1 insertion(+), 227 deletions(-)
> 
...