On 2/7/24 17:35, 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 (2): > actions: Adjust the ct_commit_nat action. > northd: Explicitly handle SNAT for ICMP need frag. > > controller/chassis.c | 8 ++ > include/ovn/actions.h | 12 ++- > include/ovn/features.h | 1 + > include/ovn/logical-fields.h | 3 + > lib/actions.c | 69 ++++++++++--- > lib/logical-fields.c | 4 + > northd/en-global-config.c | 10 ++ > northd/en-global-config.h | 1 + > northd/northd.c | 189 +++++++++++++++++++---------------- > tests/ovn-northd.at | 118 +++++++++++++++------- > tests/ovn.at | 27 ++++- > tests/system-ovn-kmod.at | 3 +- > utilities/ovn-trace.c | 2 +- > 13 files changed, 304 insertions(+), 143 deletions(-) >
Applied to main, 24.03 and 23.09, thanks for the fix! Regards, Dumitru _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
