Improve debug logging around dynamic layout loading
This will give us a better idea of how much time we're spending. Change-Id: Idbd4a8b61664d3d3af0534fb0d3f24e21808ad5a
This commit is contained in:
parent
61e4c88f0c
commit
499c3046f8
|
@ -435,12 +435,12 @@ class PipelineManager(object):
|
||||||
loader = zuul.configloader.ConfigLoader()
|
loader = zuul.configloader.ConfigLoader()
|
||||||
|
|
||||||
build_set = item.current_build_set
|
build_set = item.current_build_set
|
||||||
self.log.debug("Loading dynamic layout")
|
|
||||||
try:
|
try:
|
||||||
# First parse the config as it will land with the
|
# First parse the config as it will land with the
|
||||||
# full set of config and project repos. This lets us
|
# full set of config and project repos. This lets us
|
||||||
# catch syntax errors in config repos even though we won't
|
# catch syntax errors in config repos even though we won't
|
||||||
# actually run with that config.
|
# actually run with that config.
|
||||||
|
self.log.debug("Loading dynamic layout (phase 1)")
|
||||||
loader.createDynamicLayout(
|
loader.createDynamicLayout(
|
||||||
item.pipeline.layout.tenant,
|
item.pipeline.layout.tenant,
|
||||||
build_set.files,
|
build_set.files,
|
||||||
|
@ -450,10 +450,12 @@ class PipelineManager(object):
|
||||||
|
|
||||||
# Then create the config a second time but without changes
|
# Then create the config a second time but without changes
|
||||||
# to config repos so that we actually use this config.
|
# to config repos so that we actually use this config.
|
||||||
|
self.log.debug("Loading dynamic layout (phase 2)")
|
||||||
layout = loader.createDynamicLayout(
|
layout = loader.createDynamicLayout(
|
||||||
item.pipeline.layout.tenant,
|
item.pipeline.layout.tenant,
|
||||||
build_set.files,
|
build_set.files,
|
||||||
include_config_projects=False)
|
include_config_projects=False)
|
||||||
|
self.log.debug("Loading dynamic layout complete")
|
||||||
except zuul.configloader.ConfigurationSyntaxError as e:
|
except zuul.configloader.ConfigurationSyntaxError as e:
|
||||||
self.log.info("Configuration syntax error "
|
self.log.info("Configuration syntax error "
|
||||||
"in dynamic layout")
|
"in dynamic layout")
|
||||||
|
|
Loading…
Reference in New Issue