diff options
| author |   Martin Klozik <martinx.klozik@intel.com> | 2018-01-05 13:24:09 +0000 |
|---|---|---|
| committer | Martin Klozik <martinx.klozik@intel.com> | 2018-01-05 14:16:00 +0000 |
| commit | 58d53561bad01ae7829fdeee0c67f5dae4a9dc34 (patch) | |
| tree | eb3c8288e4dff5cf9f8a3c5622689e436dd4399a /testcases | |
| parent | b47717bb0cf09f739feb4ee9b47173c28581bf20 (diff) | |
vswitch: Improve add_phy_port error messages
In case that vSwitch can't add required number of physical
ports, then error message should be printed. Previous implementation
was not consistent. Thus error handling has been united across
all vswitches and message explicitly states the cause of the failure.
NOTE: A better solution would be a refactoring of all vSwitch
classes to follow VPP port handling and to move port related
structures and physical port check into parent vswitch class.
JIRA: VSPERF-555
Change-Id: Id09a61432ea93e261f563254829348ac61f5dc8a
Signed-off-by: Martin Klozik <martinx.klozik@intel.com>
Reviewed-by: Jose Lausuch <jalausuch@suse.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: Trevor Cooper <trevor.cooper@intel.com>
Diffstat (limited to 'testcases')
| -rw-r--r-- | testcases/testcase.py | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/testcases/testcase.py b/testcases/testcase.py index b3300b89..cf71d596 100644 --- a/testcases/testcase.py +++ b/testcases/testcase.py @@ -169,7 +169,7 @@ class TestCase(object): self._traffic['l3'] = S.getValue(self._tunnel_type.upper() + '_FRAME_L3') self._traffic['l4'] = S.getValue(self._tunnel_type.upper() + '_FRAME_L4') self._traffic['l2']['dstmac'] = S.getValue('NICS')[1]['mac'] - elif len(S.getValue('NICS')) and \ + elif len(S.getValue('NICS')) >= 2 and \ (S.getValue('NICS')[0]['type'] == 'vf' or S.getValue('NICS')[1]['type'] == 'vf'): mac1 = S.getValue('NICS')[0]['mac'] |