Skip to content
Snippets Groups Projects
Commit 5375d2c5 authored by Marcel Loose's avatar Marcel Loose :sunglasses:
Browse files

Task #2699: vdsmaker doesn't need to set self.outputs['result'] anymore,

because the framework now sets self.outputs['returncode'].
parent 5cbc23c6
No related branches found
No related tags found
No related merge requests found
...@@ -33,8 +33,7 @@ class vdsmaker(LOFARnodeTCP): ...@@ -33,8 +33,7 @@ class vdsmaker(LOFARnodeTCP):
if not os.access(executable, os.X_OK): if not os.access(executable, os.X_OK):
raise ExecutableMissing(executable) raise ExecutableMissing(executable)
cmd = [executable, clusterdesc, infile, outfile] cmd = [executable, clusterdesc, infile, outfile]
result = catch_segfaults(cmd, None, None, self.logger).returncode return catch_segfaults(cmd, None, None, self.logger).returncode
self.outputs["result"] = result
except ExecutableMissing, e: except ExecutableMissing, e:
self.logger.error("%s not found" % (e.args[0])) self.logger.error("%s not found" % (e.args[0]))
return 1 return 1
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment