Thanks Ales, it looks good to me. Acked-by: Mark Michelson <[email protected]>
On 12/6/23 05:19, Ales Musil wrote:
The ICMP need frag could be generated after routing stage when the unSNAT already happened. Add flows that will ensure that we are keeping track of the CT state and do appropriate CT nat action later on. Because the ICMP traffic is related to already existing one in this case we can use adjusted ct_commit_nat that will perform this action in SNAT zone. Ales Musil (3): actions: Adjust the ct_commit_nat action northd, controller: Add feature flag for the updated ct_commit_nat northd: Explicitly handle SNAT for ICMP need frag controller/chassis.c | 8 ++ include/ovn/actions.h | 12 ++- include/ovn/features.h | 1 + include/ovn/lex.h | 1 + include/ovn/logical-fields.h | 1 + lib/actions.c | 68 ++++++++++--- lib/lex.c | 15 +++ lib/logical-fields.c | 4 + northd/northd.c | 192 ++++++++++++++++++++--------------- northd/northd.h | 1 + tests/ovn-northd.at | 118 ++++++++++++++------- tests/ovn.at | 20 +++- tests/system-ovn-kmod.at | 3 +- utilities/ovn-trace.c | 2 +- 14 files changed, 307 insertions(+), 139 deletions(-)
_______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
