openstack/rally
Code Issues Proposed changes

Fix idle time in timeout case

Browse Source 
After switch from classess to instance of scenario classes we
stopped storeing idle time in cls. So in case of timeout, code like:
> cls.idle_time will produce error

Change-Id: I54b6f87a45560b0c98ce6eeb60d27decfa06ad5b
Closes-bug: #1275724
This commit is contained in:
Boris Pavlovic
2014-02-03 16:59:23 +04:00
parent 5f5d90d08c
commit a89df76c17
1 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
rally/benchmark/runner.py
View File

@@ -153,7 +153,7 @@ class ScenarioRunner(object):
try:
result = iter_result.next(timeout)
except multiprocessing.TimeoutError as e:
result = {"time": timeout, "idle_time": cls.idle_time,
result = {"time": timeout, "idle_time": 0,
"error": utils.format_exc(e)}
results.append(result)
@@ -180,7 +180,7 @@ class ScenarioRunner(object):
try:
result = iter_result.next(timeout)
except multiprocessing.TimeoutError as e:
result = {"time": timeout, "idle_time": cls.idle_time,
result = {"time": timeout, "idle_time": 0,
"error": utils.format_exc(e)}
results_queue.append(result)
@@ -209,7 +209,7 @@ class ScenarioRunner(object):
try:
result = async_result.get()
except multiprocessing.TimeoutError as e:
result = {"time": timeout, "idle_time": cls.idle_time,
result = {"time": timeout, "idle_time": 0,
"error": utils.format_exc(e)}
results.append(result)