zuul
/
zuul
Code Issues Proposed changes

Don't log file contents when loading dynamic layouts 

These files can get quite large.

Change-Id: I7d6b25f6767291260d567f9ff69735c1e27416c5
This commit is contained in:
James E. Blair 2017-09-23 07:44:10 -07:00
parent d275039481
commit 6e00a8596d
1 changed files with 3 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
zuul/manager/__init__.py
View File

@ -435,7 +435,7 @@ class PipelineManager(object):
loader = zuul.configloader.ConfigLoader()
build_set = item.current_build_set
self.log.debug("Load dynamic layout with %s" % build_set.files)
self.log.debug("Loading dynamic layout")
try:
# First parse the config as it will land with the
# full set of config and project repos. This lets us
@ -456,13 +456,11 @@ class PipelineManager(object):
include_config_projects=False)
except zuul.configloader.ConfigurationSyntaxError as e:
self.log.info("Configuration syntax error "
"in dynamic layout %s" %
build_set.files)
"in dynamic layout")
item.setConfigError(str(e))
return None
except Exception:
self.log.exception("Error in dynamic layout %s" %
build_set.files)
self.log.exception("Error in dynamic layout")
item.setConfigError("Unknown configuration error")
return None
return layout