<!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">
<html>
<head>
<meta content="text/html;charset=UTF-8" http-equiv="Content-Type">
</head>
<body bgcolor="#ffffcc" text="#000066">
<tt>Signed-off-by: Marcos Matsunaga <a class="moz-txt-link-rfc2396E" href="mailto:Marcos.Matsunaga@oracle.com"><Marcos.Matsunaga@oracle.com></a></tt><br>
<pre class="moz-signature" cols="72">Regards,
Marcos Eduardo Matsunaga
Oracle USA
Linux Engineering
“The statements and opinions expressed here are my own and do not
necessarily represent those of Oracle Corporation.”
</pre>
<br>
<br>
Tristan Ye wrote:
<blockquote
cite="mid:1231472550-26612-2-git-send-email-tristan.ye@oracle.com"
type="cite">
<pre wrap="">Once mpi_run by os.waitpid() failed after a os.error exception captured, we'd
better print its corresponding error msg and return error code to caller to decide
if the tests succeed or not.
Signed-off-by: Tristan Ye <a class="moz-txt-link-rfc2396E" href="mailto:tristan.ye@oracle.com"><tristan.ye@oracle.com></a>
---
programs/python_common/o2tf.py | 7 ++++---
1 files changed, 4 insertions(+), 3 deletions(-)
diff --git a/programs/python_common/o2tf.py b/programs/python_common/o2tf.py
index 53d7865..434d3c4 100644
--- a/programs/python_common/o2tf.py
+++ b/programs/python_common/o2tf.py
@@ -234,7 +234,7 @@ so just do a sanity check here to test if all nodes are available.
except os.error,inst:
printlog(str(inst), logfile, 0, '')
- pass
+ return -1
#
# StartMPI is used by :
# - o2tf.py
@@ -343,8 +343,9 @@ def openmpi_run(DEBUGON, nproc, cmd, nodes, remote_sh, logfile, w_flag):
else:
os.waitpid(pid, 0)
- except os.error:
- pass
+ except os.error,inst:
+ printlog(str(inst), logfile, 0, '')
+ return -1
#
# lamexec is used by :
</pre>
</blockquote>
</body>
</html>