diff options
author | Christian Trautman <ctrautma@redhat.com> | 2016-04-20 15:30:42 -0400 |
---|---|---|
committer | Christian Trautman <ctrautma@redhat.com> | 2016-05-04 09:10:46 -0400 |
commit | 47875a6f01fa9dc1738d4c4a29ad75e12e251077 (patch) | |
tree | cc53a18eb151cafb200493057b0bfc89c870c44d /core/vswitch_controller_pvp.py | |
parent | 25969600ac9508ecc54a25d7b0f628e0713a82a2 (diff) |
bidirectional value: Fix inconsistency for bidir param
Fixes bi-directional param and enforces string type in
testcase configuration. Modifies all instance where bi
directional value is used to string.
* Changes documentation to specify value type as needed
* Changes traffic defaults to use string for bidir value
* Fixes testcases configuration files to use bidir as str
* Change testcase conf documentation to note enforcement
of string type
* Sets bidir to title format to keep consitency in traffic
profile when passed to traffic generators
* Fix pvp/pvvp check for bidir value
* Raise exception if testcase configuration is not set
as str for bidir value
* Change Xena bi-dir check to string compare instead of
typecasting to boolean to compare.
JIRA: VSPERF-282
Change-Id: I7c91fcd7333a1d1e1371bfd31700c01d7da268b5
Signed-off-by: Christian Trautman <ctrautma@redhat.com>
Diffstat (limited to 'core/vswitch_controller_pvp.py')
-rw-r--r-- | core/vswitch_controller_pvp.py | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/core/vswitch_controller_pvp.py b/core/vswitch_controller_pvp.py index 0c98cc7f..a4f61961 100644 --- a/core/vswitch_controller_pvp.py +++ b/core/vswitch_controller_pvp.py @@ -77,7 +77,7 @@ class VswitchControllerPVP(IVswitchController): self._vswitch.add_flow(bridge, flow1) self._vswitch.add_flow(bridge, flow2) - if self._traffic['bidir']: + if self._traffic['bidir'] == 'True': flow3 = add_ports_to_flow(flow_template, phy2_number, vport2_number) flow4 = add_ports_to_flow(flow_template, vport1_number, |