diff options
| author |   Yaroslav Brustinov <ybrustin@cisco.com> | 2016-12-23 00:27:31 +0200 |
|---|---|---|
| committer | Yaroslav Brustinov <ybrustin@cisco.com> | 2016-12-23 00:27:31 +0200 |
| commit | 63c6f35a263a135ebd44334ab4c9b0e22424bb5b (patch) | |
| tree | d68ab7da49462216190b1af3103fd7712968ac73 /scripts/automation | |
| parent | 9c61c44f55b0b19be441c2a9561a93f81f8e7a41 (diff) | |
Regression stateful: division by zero due to rounding of small numbers, only warn for too high BW, not fail. (will be noticed by failured of queue full etc.)
Change-Id: I4ecfb13032a3d8cb27fb224fb5ed8dd4e10f68a2
Signed-off-by: Yaroslav Brustinov <ybrustin@cisco.com>
Diffstat (limited to 'scripts/automation')
| -rwxr-xr-x | scripts/automation/regression/stateful_tests/trex_general_test.py | 15 |
1 files changed, 7 insertions, 8 deletions
diff --git a/scripts/automation/regression/stateful_tests/trex_general_test.py b/scripts/automation/regression/stateful_tests/trex_general_test.py index 088bd97d..f6d2b917 100755 --- a/scripts/automation/regression/stateful_tests/trex_general_test.py +++ b/scripts/automation/regression/stateful_tests/trex_general_test.py @@ -223,16 +223,15 @@ class CTRexGeneral_Test(unittest.TestCase): self.fail('TRex did not reach warm-up situtaion. Results are not valid.') # check that BW is not much more than expected - trex_exp_bps = int(trex_res.get_expected_tx_rate().get('m_tx_expected_bps') / 1e6) - trex_cur_bps = int(max(trex_res.get_value_list('trex-global.data.m_tx_bps')) / 1e6) + trex_exp_bps = trex_res.get_expected_tx_rate().get('m_tx_expected_bps') / 1e6 + trex_cur_bps = max(trex_res.get_value_list('trex-global.data.m_tx_bps')) / 1e6 - if trex_exp_bps is None: - self.fail('Expected rate is None!') - if trex_cur_bps is None: - self.fail('Current rate is None!') + assert trex_exp_bps > 0, 'Expected BPS is zero: %s' % trex_exp_bps - if trex_exp_bps * 1.05 < trex_cur_bps: - self.fail('Got BW (%sMbps) that is %s%% more than expected (%sMbps)!' % (trex_cur_bps, round(100.0 * trex_cur_bps / trex_exp_bps - 100, 2), trex_exp_bps)) + if trex_exp_bps * 1.05 + 10 < trex_cur_bps: + msg = 'Got BW (%gMbps) that is %g%% more than expected (%gMbps)!' % (round(trex_cur_bps, 2), round(100.0 * trex_cur_bps / trex_exp_bps - 100, 2), round(trex_exp_bps, 2)) + print('WARNING: %s' % msg) + #self.fail(msg) # check TRex number of drops trex_tx_pckt = trex_res.get_last_value("trex-global.data.m_total_tx_pkts") |