bugfix: Define _EXECUTED_PIDS in config file

Add definition of internal _EXECUTED_PIDS list into generic
configuration file.

JIRA: VSPERF-556

Change-Id: I6c237cb563c0a37bf806561d1f841587796399b8
Signed-off-by: Martin Klozik <martinx.klozik@intel.com>

diff --git a/conf/00_common.conf b/conf/00_common.conf
index 4c25b0b..279c67b 100644 (file)
--- a/conf/00_common.conf
+++ b/conf/00_common.conf
@@ -90,6 +90,9 @@ PATHS = {}
 # shell command to use when running commands through Pexpect
 SHELL_CMD = ['/bin/bash', '-c']
 
+# internal list to keep track of PIDs of jobs executed by vsperf
+_EXECUTED_PIDS = []
+
 # ############################
 # Logging configuration
 # ############################

diff --git a/tools/tasks.py b/tools/tasks.py
index 18f4d71..4e03f85 100644 (file)
--- a/tools/tasks.py
+++ b/tools/tasks.py
@@ -117,11 +117,8 @@ def run_task(cmd, logger, msg=None, check_error=False):
 def update_pids(pid):
     """update list of running pids, so they can be terminated at the end
     """
-    try:
-        pids = settings.getValue('_EXECUTED_PIDS')
-        pids.append(pid)
-    except AttributeError:
-        pids = [pid]
+    pids = settings.getValue('_EXECUTED_PIDS')
+    pids.append(pid)
     settings.setValue('_EXECUTED_PIDS', pids)
 
 def run_background_task(cmd, logger, msg):