Skip to content

Commit

Permalink
system-traffic: Remove datapath specific tests and macro.
Browse files Browse the repository at this point in the history
We generally try to keep the testsuite independent of the underlying
datapath. This patch removes the datapath-specific tests and macros.

Tested-at: https://travis-ci.org/williamtu/ovs-travis/builds/141642065
Signed-off-by: William Tu <[email protected]>
Signed-off-by: Joe Stringer <[email protected]>
  • Loading branch information
williamtu authored and joestringer committed Jul 15, 2016
1 parent 6a9d35d commit 3041e1f
Show file tree
Hide file tree
Showing 3 changed files with 1 addition and 26 deletions.
7 changes: 0 additions & 7 deletions tests/system-kmod-macros.at
Original file line number Diff line number Diff line change
Expand Up @@ -66,10 +66,3 @@ m4_define([CHECK_CONNTRACK],
on_exit 'ovstest test-netlink-conntrack flush'
]
)

# CHECK_KERNEL_DP, CHECK_USER_DP
#
# Ignore the CHECK_USER_DP and execute the CHECK_KERNEL_DP
#
m4_define([CHECK_KERNEL_DP], [$1])
m4_define([CHECK_USER_DP], [])
13 changes: 1 addition & 12 deletions tests/system-traffic.at
Original file line number Diff line number Diff line change
Expand Up @@ -334,14 +334,12 @@ dnl SLOW_ACTION test1: check datapatch actions
AT_CHECK([ovs-ofctl del-flows br0])
AT_CHECK([ovs-ofctl add-flows br0 flows.txt])

CHECK_KERNEL_DP(
AT_CHECK([ovs-appctl ofproto/trace system 'in_port(2),eth(src=e6:66:c1:11:11:11,dst=e6:66:c1:22:22:22),eth_type(0x0800),ipv4(src=192.168.0.1,dst=192.168.0.2,proto=6,tos=4,ttl=128,frag=no),tcp(src=8,dst=9)'], [0], [stdout])
AT_CHECK([ovs-appctl ofproto/trace br0 "in_port=1,dl_type=0x800,dl_src=e6:66:c1:11:11:11,dl_dst=e6:66:c1:22:22:22,nw_src=192.168.0.1,nw_dst=192.168.0.2,nw_proto=6,tp_src=8,tp_dst=9"], [0], [stdout])
AT_CHECK([tail -3 stdout], [0],
[Datapath actions: trunc(100),3,5,trunc(100),3,trunc(100),5,3,trunc(200),5,trunc(65535),3
This flow is handled by the userspace slow path because it:
- Uses action(s) not supported by datapath.
])
)

dnl SLOW_ACTION test2: check actual packet truncate
AT_CHECK([ovs-ofctl del-flows br0])
Expand Down Expand Up @@ -458,15 +456,6 @@ dnl SLOW_ACTION test1: check datapatch actions
AT_CHECK([ovs-ofctl del-flows br0])
AT_CHECK([ovs-ofctl add-flows br0 flows.txt])

CHECK_KERNEL_DP(
AT_CHECK([ovs-appctl ofproto/trace system 'in_port(5),eth(src=e6:66:c1:11:11:11,dst=e6:66:c1:22:22:22),eth_type(0x0800),ipv4(src=192.168.0.1,dst=192.168.0.2,proto=17,tos=4,ttl=128,frag=no),udp(src=8,dst=9)'], [0], [stdout])
AT_CHECK([tail -3 stdout], [0],
[Datapath actions: trunc(100),set(tunnel(dst=172.31.1.1,ttl=64,flags(df))),4
This flow is handled by the userspace slow path because it:
- Uses action(s) not supported by datapath.
])
)

dnl SLOW_ACTION test2: check actual packet truncate
AT_CHECK([ovs-ofctl del-flows br0])
AT_CHECK([ovs-ofctl add-flows br0 flows.txt])
Expand Down
7 changes: 0 additions & 7 deletions tests/system-userspace-macros.at
Original file line number Diff line number Diff line change
Expand Up @@ -66,10 +66,3 @@ m4_define([CONFIGURE_VETH_OFFLOADS],
m4_define([CHECK_CONNTRACK],
[AT_SKIP_IF(true)]
)

# CHECK_KERNEL_DP, CHECK_USER_DP
#
# Ignore the CHECK_KERNEL_DP and execute the CHECK_USER_DP
#
m4_define([CHECK_KERNEL_DP], [])
m4_define([CHECK_USER_DP], [$1])

0 comments on commit 3041e1f

Please sign in to comment.