ovs: Change default bridge names

Default names of OVS bridges used by vsperf were changed
to avoid accidental collision with 3rd party bridges
(e.g. linux bridges) already configured at DUT.
Step driven testcases, which extend existing deployment
scenarios, were updated to use appropriate configuration
options instead of hardcoded bridge names.

JIRA: VSPERF-580

Change-Id: I8c7eee6d8f6532485ec3a2cc06f26515e665fb7e
Signed-off-by: Martin Klozik <martin.klozik@tieto.com>
Reviewed-by: Al Morton <acmorton@att.com>
Reviewed-by: Christian Trautman <ctrautma@redhat.com>
Reviewed-by: Sridhar Rao <sridhar.rao@spirent.com>
Reviewed-by: Richard Elias <richard.elias@tieto.com>

1 files changed, 23 insertions, 21 deletions

diff --git a/conf/integration/01b_dpdk_regression_tests.conf b/conf/integration/01b_dpdk_regression_tests.conf
index ed9c1e60..abc56c28 100644
--- a/conf/integration/01b_dpdk_regression_tests.conf
+++ b/conf/integration/01b_dpdk_regression_tests.conf
@@ -1,4 +1,4 @@
-# Copyright 2017-2018 Intel Corporation.
+# Copyright 2017-2018 Intel Corporation., Tieto
 #
 # Licensed under the Apache License, Version 2.0 (the "License");
 # you may not use this file except in compliance with the License.
@@ -341,14 +341,14 @@ INTEGRATION_TESTS = INTEGRATION_TESTS + [
                         # frame loss with small packets should be detected
                         ['tools', 'assert', '#STEP[-1][0]["frame_loss_percent"] > 10'],
                         # delete phy ports so they can be created with flow control
-                        ['vswitch', 'del_flow', 'br0', {}],
-                        ['vswitch', 'del_port', 'br0', 'dpdk0'],
-                        ['vswitch', 'del_port', 'br0', 'dpdk1'],
+                        ['vswitch', 'del_flow', '$VSWITCH_BRIDGE_NAME', {}],
+                        ['vswitch', 'del_port', '$VSWITCH_BRIDGE_NAME', 'dpdk0'],
+                        ['vswitch', 'del_port', '$VSWITCH_BRIDGE_NAME', 'dpdk1'],
                         # turn on flow control
-                        ['tools', 'exec_shell', 'sudo $TOOLS["ovs-vsctl"] add-port br0 dpdk0 -- set Interface dpdk0 type=dpdk options:dpdk-devargs=$NICS[0]["pci"] options:rx-flow-ctrl=true'],
-                        ['tools', 'exec_shell', 'sudo $TOOLS["ovs-vsctl"] add-port br0 dpdk1 -- set Interface dpdk1 type=dpdk options:dpdk-devargs=$NICS[1]["pci"] options:rx-flow-ctrl=true'],
-                        ['vswitch', 'add_flow', 'br0', {'in_port': '3', 'actions': ['output:4'], 'idle_timeout': '0'}],
-                        ['vswitch', 'add_flow', 'br0', {'in_port': '4', 'actions': ['output:3'], 'idle_timeout': '0'}],
+                        ['tools', 'exec_shell', 'sudo $TOOLS["ovs-vsctl"] add-port $VSWITCH_BRIDGE_NAME dpdk0 -- set Interface dpdk0 type=dpdk options:dpdk-devargs=$NICS[0]["pci"] options:rx-flow-ctrl=true'],
+                        ['tools', 'exec_shell', 'sudo $TOOLS["ovs-vsctl"] add-port $VSWITCH_BRIDGE_NAME dpdk1 -- set Interface dpdk1 type=dpdk options:dpdk-devargs=$NICS[1]["pci"] options:rx-flow-ctrl=true'],
+                        ['vswitch', 'add_flow', '$VSWITCH_BRIDGE_NAME', {'in_port': '3', 'actions': ['output:4'], 'idle_timeout': '0'}],
+                        ['vswitch', 'add_flow', '$VSWITCH_BRIDGE_NAME', {'in_port': '4', 'actions': ['output:3'], 'idle_timeout': '0'}],
                         ['tools', 'exec_shell', 'sudo $TOOLS["ovs-vsctl"] show'],
                         ['trafficgen', 'send_traffic', {}],
                         ['trafficgen', 'get_results'],
@@ -668,15 +668,17 @@ INTEGRATION_TESTS = INTEGRATION_TESTS + [
         "TestSteps": [
                         # send traffic to verify correct PVP configuration
                         ['trafficgen', 'send_traffic', {}],
-                        ['vswitch', 'dump_flows', 'br0'],
+                        ['vswitch', 'dump_flows', '$VSWITCH_BRIDGE_NAME'],
                         # restart vswitchd, ovsdb is kept untouched, so ovs configuration
                         # (except flows) will be restored
                         ['vswitch', 'restart'],
-                        ['vswitch', 'del_flow', 'br0'],
-                        ['vswitch', 'add_flow', 'br0', {'in_port': '1', 'actions': ['output:3'], 'idle_timeout': '0'}],
-                        ['vswitch', 'add_flow', 'br0', {'in_port': '3', 'actions': ['output:1'], 'idle_timeout': '0'}],
-                        ['vswitch', 'add_flow', 'br0', {'in_port': '2', 'actions': ['output:4'], 'idle_timeout': '0'}],
-                        ['vswitch', 'add_flow', 'br0', {'in_port': '4', 'actions': ['output:2'], 'idle_timeout': '0'}],
+                        ['vswitch', 'del_flow', '$VSWITCH_BRIDGE_NAME', {}],
+                        ['vswitch', 'dump_flows', '$VSWITCH_BRIDGE_NAME'],
+                        ['vswitch', 'add_flow', '$VSWITCH_BRIDGE_NAME', {'in_port': '1', 'actions': ['output:3'], 'idle_timeout': '0'}],
+                        ['vswitch', 'add_flow', '$VSWITCH_BRIDGE_NAME', {'in_port': '3', 'actions': ['output:1'], 'idle_timeout': '0'}],
+                        ['vswitch', 'add_flow', '$VSWITCH_BRIDGE_NAME', {'in_port': '2', 'actions': ['output:4'], 'idle_timeout': '0'}],
+                        ['vswitch', 'add_flow', '$VSWITCH_BRIDGE_NAME', {'in_port': '4', 'actions': ['output:2'], 'idle_timeout': '0'}],
+                        ['vswitch', 'dump_flows', '$VSWITCH_BRIDGE_NAME'],
                         # send traffic to verify that OVS works correctly after restart
                         ['trafficgen', 'send_traffic', {}],
                      ],
@@ -715,13 +717,13 @@ INTEGRATION_TESTS = INTEGRATION_TESTS + [
             "VSWITCH_VHOSTUSER_SERVER_MODE" : True,
         },
         "TestSteps": [
-                        ['vswitch', 'add_switch', 'br0'],
+                        ['vswitch', 'add_switch', '$VSWITCH_BRIDGE_NAME'],
                         ['tools', 'exec_shell', 'sudo $TOOLS["ovs-vsctl"] set Open_vSwitch . '
                          'other_config:vhost-sock-dir=test_dir'],
                         # enforce vswitchd to read new configuration
                         ['vswitch', 'restart'],
                         ['tools', 'exec_shell', 'sudo mkdir $TOOLS["ovs_var_tmp"]/test_dir'],
-                        ['vswitch', 'add_vport', 'br0'],
+                        ['vswitch', 'add_vport', '$VSWITCH_BRIDGE_NAME'],
                         ['tools', 'exec_shell', 'ls -1 $TOOLS["ovs_var_tmp"]/test_dir',
                          '|dpdkvhostuser0'],
                         ['tools', 'assert', 'len(#STEP[-1])'],
@@ -1294,7 +1296,7 @@ _OVSDPDK_RATE_confirm_multiple_rate_limit_setup = \
         # check that traffic rate is no longer limited
         ['trafficgen', 'get_results'],
         ['tools', 'assert', 'int(#STEP[-1][0]["throughput_rx_mbps"])>500'],
-        ['vswitch', 'dump_flows', 'br0'],
+        ['vswitch', 'dump_flows', '$VSWITCH_BRIDGE_NAME'],
     ]
 
 INTEGRATION_TESTS = INTEGRATION_TESTS + [
@@ -1600,8 +1602,8 @@ INTEGRATION_TESTS = INTEGRATION_TESTS + [
         "Parameters" : {},
         "TestSteps": [
                          # Setup switch,port and logs
-                        ['vswitch', 'add_switch', 'br0'],
-                        ['vswitch', 'add_vport', 'br0'],
+                        ['vswitch', 'add_switch', '$VSWITCH_BRIDGE_NAME'],
+                        ['vswitch', 'add_vport', '$VSWITCH_BRIDGE_NAME'],
                         ['#LOG_MARK', 'tools', 'exec_shell',
                          'echo $((1+`wc -l $_OVSDPDK_VSWITCH_LOG | cut -d" " -f1`))', '(\d+)'],
 
@@ -1630,8 +1632,8 @@ INTEGRATION_TESTS = INTEGRATION_TESTS + [
         "Parameters" : {},
         "TestSteps": [
                          # Setup switch,port and logs
-                        ['vswitch', 'add_switch', 'br0'],
-                        ['vswitch', 'add_vport', 'br0'],
+                        ['vswitch', 'add_switch', '$VSWITCH_BRIDGE_NAME'],
+                        ['vswitch', 'add_vport', '$VSWITCH_BRIDGE_NAME'],
                         ['#LOG_MARK', 'tools', 'exec_shell',
                          'echo $((1+`wc -l $_OVSDPDK_VSWITCH_LOG | cut -d" " -f1`))', '(\d+)'],