diff mbox

[ovs-dev] system-ovn.at: Add test for ping other router's port on distributed router

Message ID CAPWQB7EATJKJFiF=buTpoD61inJ=1T2NXJj=S=_aDfrQtLwidw@mail.gmail.com
State Not Applicable
Headers show

Commit Message

Joe Stringer April 18, 2017, 9:51 p.m. UTC 
On 18 April 2017 at 04:49, Guoshuai Li <ligs@dtdream.com> wrote:
> Signed-off-by: Guoshuai Li <ligs@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:

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.
diff mbox

Patch

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()