diff options
author | Dimitrios Markou <mardim@intracom-telecom.com> | 2018-07-04 11:49:54 +0300 |
---|---|---|
committer | Dimitrios Markou <mardim@intracom-telecom.com> | 2018-07-06 07:03:16 +0000 |
commit | 5821098118bb97ce9d1a302bb8f19b615ebb01cd (patch) | |
tree | 3a7a649b285636617f861f50e723fdf0cdfc2227 | |
parent | a99613376daa0656bcd0b24d9c56661ff15c8916 (diff) |
[Bug fix] Execute teardown if SLA is set to "assert"
In Duration runner if SLA is set to "assert" and a SLAValidationError
happens then a teardown is needed to avoid garbages in the environment.
JIRA: YARDSTICK-1291
Change-Id: I9056b0da3e7e496171ed42430ffc340b5c68f108
Signed-off-by: Dimitrios Markou <mardim@intracom-telecom.com>
(cherry picked from commit 30fc9f545d526fdbd69af483b0bad0ea781d648c)
-rw-r--r-- | yardstick/benchmark/runners/duration.py | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/yardstick/benchmark/runners/duration.py b/yardstick/benchmark/runners/duration.py index 60f1fa536..14fd8bb47 100644 --- a/yardstick/benchmark/runners/duration.py +++ b/yardstick/benchmark/runners/duration.py @@ -74,6 +74,7 @@ def _worker_process(queue, cls, method_name, scenario_cfg, except y_exc.SLAValidationError as error: # SLA validation failed in scenario, determine what to do now if sla_action == "assert": + benchmark.teardown() raise elif sla_action == "monitor": LOG.warning("SLA validation failed: %s", error.args) |