From: Per Holmgren Date: Thu, 17 Dec 2015 15:15:50 +0000 (+0100) Subject: To avoid breaking tests when monitor is set X-Git-Tag: brahmaputra.1.0~116 X-Git-Url: https://gerrit.opnfv.org/gerrit/gitweb?a=commitdiff_plain;h=edb23ec25362d2c0c43e124c03fe3d4fe7f1e4f0;p=yardstick.git To avoid breaking tests when monitor is set Two runners, Arithemic and Sequence, broke running tests even if SLA "monitor" was set. Changed these two runners to work in similar fashion as the other two, i.e. that only None can break. "assert" still asserts, and "monitor" just logs but then continues. Change-Id: Ic441761cad8747e5ecfcd0babe719c1aa23981e7 JIRA: - --- diff --git a/yardstick/benchmark/runners/arithmetic.py b/yardstick/benchmark/runners/arithmetic.py index b1446e01c..7e516ea1e 100755 --- a/yardstick/benchmark/runners/arithmetic.py +++ b/yardstick/benchmark/runners/arithmetic.py @@ -124,7 +124,7 @@ def _worker_process(queue, cls, method_name, scenario_cfg, sequence += 1 - if errors: + if (errors and sla_action is None): break benchmark.teardown() diff --git a/yardstick/benchmark/runners/sequence.py b/yardstick/benchmark/runners/sequence.py index 47708fc5e..b5fae37ad 100644 --- a/yardstick/benchmark/runners/sequence.py +++ b/yardstick/benchmark/runners/sequence.py @@ -96,7 +96,7 @@ def _worker_process(queue, cls, method_name, scenario_cfg, sequence += 1 - if errors or aborted.is_set(): + if (errors and sla_action is None) or aborted.is_set(): break benchmark.teardown()