On 8/27/21 2:41 PM, Flavio Fernandes wrote:
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?
Agreed.
Acked-by: Flavio Fernandes <[email protected] <mailto:[email protected]>>
On Aug 27, 2021, at 11:29 AM, Mark Michelson <[email protected]
<mailto:[email protected]>> 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
<https://bugzilla.redhat.com/show_bug.cgi?id=1992012>
Signed-off-by: Mark Michelson <[email protected]
<mailto:[email protected]>>
---
include/ovn/actions.h | 1 -
lib/actions.c | 31 ------
tests/ovn.at <http://ovn.at> | 2 +-
tests/system-common-macros.at <http://system-common-macros.at> | 4 -
tests/system-ovn.at <http://system-ovn.at> | 190
----------------------------------
5 files changed, 1 insertion(+), 227 deletions(-)
...
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
