On 18 April 2017 at 04:49, Guoshuai Li <l...@dtdream.com> wrote: > Signed-off-by: Guoshuai Li <l...@dtdream.com> > ---
Hi Guoshuai, Thanks for the patch. Is this intended for debugging some current issue? I can't get these new tests to pass "make check-kmod". Furthermore, for "make check-system-userspace", the "datapath - SNAT and UNSNAT" test needs to check for CT, NAT features, eg using the following incremental: diff --git a/tests/system-traffic.at b/tests/system-traffic.at index 295e6062579e..6f637f257b75 100644 --- a/tests/system-traffic.at +++ b/tests/system-traffic.at @@ -3680,6 +3680,8 @@ OVS_TRAFFIC_VSWITCHD_STOP(["/dropping VLAN \(0\|300\) packet received on dot1q-t AT_CLEANUP AT_SETUP([datapath - SNAT and UNSNAT]) +CHECK_CONNTRACK() +CHECK_CONNTRACK_NAT() OVS_TRAFFIC_VSWITCHD_START() AT_CHECK([ovs-ofctl add-flow br0 "table=0, priority=100,in_port=1,ip,nw_dst=20.0.0.2 actions=dec_ttl(),mod_dl_src:00:00:02:01:02:01,mod_dl_dst:00:00:02:01:02:02,resubmit(,1)"]) <snip> > -- > 2.10.1.windows.1 > > This patch is used to analyze "ovn: unsnat handling error for Distributed > Gateway" problems: > > https://mail.openvswitch.org/pipermail/ovs-dev/2017-April/331033.html Ah :-) Usually this appears immediately below a "---" after the commit message. I'll mark this in patchwork as 'Deferred'. When a fix is supplied, we could apply this patch at the same time. _______________________________________________ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev