diff mbox

[ovs-dev] system-traffic: add a bonding test case

Message ID CAPWQB7E43cqjQ=yXVJyQKnop2Yy6GXdTb8rVOTcnmgpdg4AV6g@mail.gmail.com
State Not Applicable
Headers show

Commit Message

Joe Stringer Aug. 31, 2016, 6:15 p.m. UTC 
On 30 August 2016 at 13:45, Lance Richardson <lrichard@redhat.com> wrote:
> Add a test case to check connectivity over an OVS bond, using a
> Linux bond over veth interfaces.
>
> Also added a new macro "ADD_VETH_BOND", modeled after "ADD_VETH",
> in anticipation of future additional bonding test cases.
>
> Signed-off-by: Lance Richardson <lrichard@redhat.com>

Thanks, this passes on all relevant platforms I could test on. I
applied the following minor diff and applied this patch to master:
diff mbox

Patch

diff --git a/tests/system-common-macros.at b/tests/system-common-macros.at
index 252ec8944382..1dfdcf9ea789 100644
--- a/tests/system-common-macros.at
+++ b/tests/system-common-macros.at
@@ -90,9 +90,9 @@  m4_define([ADD_VETH],

# ADD_VETH_BOND([ports], [namespace], [ovs-br], [bond], [mode], [ip_addr])
#
-# Add a set of veth port pairs. Ports named in the list 'ports' will
be added to
-# name space 'namespace', and the corresponding port names, prefixed by 'ovs-'
-# will be included in an OVS bond 'bond' which is added to bridge 'ovs-br'.
+# Add a set of veth port pairs. Ports named in the list 'ports' will be added
+# to 'namespace', and the corresponding port names, prefixed by 'ovs-' will
+# be included in an OVS bond 'bond' which is added to bridge 'ovs-br'.
#
# The 'bond' in 'namespace' will be brought up with static IP address
# with 'ip_addr' in CIDR notation.
diff --git a/tests/system-traffic.at b/tests/system-traffic.at
index aec5999b7d67..eaf4aba13869 100644
--- a/tests/system-traffic.at
+++ b/tests/system-traffic.at
@@ -150,8 +150,6 @@  NS_CHECK_EXEC([at_ns0], [ping -s 3200 -q -c 3 -i
0.3 -w 2 10.1.1.2 | FORMAT_PING
3 packets transmitted, 3 received, 0% packet loss, time 0ms
])

-ovs-appctl dpif/dump-flows br0
-
OVS_TRAFFIC_VSWITCHD_STOP
AT_CLEANUP