Commit f4488133 authored by Samuel GAIST's avatar Samuel GAIST

[execution][subprocess] Update the status variable per container for stderr handling

Currently only the status of the main algorithm container
was taken into account to gather the stderr output of all
other containers. This patch changes this so that if
something fails but the main containers ends successfully
the other containers output can still be analysed.
parent e10dbe66
......@@ -436,6 +436,7 @@ class SubprocessExecutor(RemoteExecutor):
databases_process.terminate()
databases_process.communicate()
status = databases_process.returncode
with open(db_info['stdout'].name, 'r') as f:
retval['stdout'] += '\n' + f.read()
......@@ -448,6 +449,7 @@ class SubprocessExecutor(RemoteExecutor):
logger.debug("Stopping loop process")
loop_algorithm_process.terminate()
loop_algorithm_process.communicate()
status = loop_algorithm_process.returncode
with open(loop_algorithm_process_stdout.name, 'r') as f:
retval['stdout'] += '\n' + f.read()
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment