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

be consistent - log parsing is easy

Browse source 
Signed-off-by: Nico Schottelius <nico@brief.schottelius.org>
This commit is contained in:
Nico Schottelius 2012-03-10 00:25:13 +01:00
parent 1ca42f5dd5
commit dc355e7aa2
1 changed files with 1 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
bin/cdist
View file

@ -132,8 +132,6 @@ def configinstall(args, mode):
try: try:
configinstall_onehost(host, args, mode, parallel=False) configinstall_onehost(host, args, mode, parallel=False)
except cdist.Error as e: except cdist.Error as e:
# FIXME: save the error and display at the end?
# Would be non-consistent to parallel mode
failed_hosts.append(host) failed_hosts.append(host)
# Catch errors in parallel mode when joining # Catch errors in parallel mode when joining
@ -171,6 +169,7 @@ def configinstall_onehost(host, args, mode, parallel):
context.cleanup() context.cleanup()
except cdist.Error as e: except cdist.Error as e:
# We are running in our own process here, need to sys.exit!
if parallel: if parallel:
log.error(e) log.error(e)
sys.exit(1) sys.exit(1)