On 1/5/23 13:07, Eelco Chaudron wrote: > The dpif_execute_helper_cb() function is supposed to add the > OVS_ACTION_ATTR_SET(OVS_KEY_ATTR_TUNNEL()) action to the > list of actions when passing it down to the kernel. > > This function was only checking if the IPv4 destination > address was set, not both. This patch fixes this, including > a datapath testcase. > > Fixes: 076caa2fb077 ("ofproto: Meter translation.") > Signed-off-by: Eelco Chaudron <echau...@redhat.com> > --- > > v2: Changed capture file name to txt as it's was not in pcap format. > v3: Using ovs-pcap rather than relying on the tcpdump output. > v4: Exclude udp checksum from result comparison. > > lib/dpif.c | 2 +- > tests/system-traffic.at | 48 > +++++++++++++++++++++++++++++++++++++++++++++++ > 2 files changed, 49 insertions(+), 1 deletion(-)
Thanks! Applied and backported down to 2.13. Best regards, Ilya Maximets. _______________________________________________ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev