moep/cdist
1
0
Fork 0
Code Issues Pull requests Projects Releases Wiki Activity

add missing return_output=True argument when fetching script content

Browse source 
Signed-off-by: Steven Armstrong <steven@icarus.ethz.ch>
This commit is contained in:
Steven Armstrong 2011-10-14 23:52:05 +02:00
parent 325a13088c
commit b5b7d8e727
2 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
lib/cdist/exec/local.py
View file

@ -122,7 +122,7 @@ class Local(object):
else:
subprocess.check_call(command, env=env)
except subprocess.CalledProcessError as error:
script_content = self.run(["cat", script])
script_content = self.run(["cat", script], return_output=True)
self.log.error("Code that raised the error:\n%s", script_content)
raise LocalScriptError(script, command, script_content)
except EnvironmentError as error:

2
lib/cdist/exec/remote.py
View file

@ -148,7 +148,7 @@ class Remote(object):
else:
subprocess.check_call(command)
except subprocess.CalledProcessError as error:
script_content = self.run(["cat", script])
script_content = self.run(["cat", script], return_output=True)
self.log.error("Code that raised the error:\n%s", script_content)
raise RemoteScriptError(script, command, script_content)
except EnvironmentError as error: