Merge "Add catch-all for property errors in implicit dependencies"

This commit is contained in:
Zuul 2018-05-02 10:54:15 +00:00 committed by Gerrit Code Review
commit f9e239ef26
1 changed files with 5 additions and 1 deletions

View File

@ -514,7 +514,11 @@ class Stack(collections.Mapping):
try:
res.add_dependencies(deps)
except Exception as exc:
if not ignore_errors:
# Always ignore ValueError/TypeError, as they're likely to
# have come from trying to read invalid property values that
# haven't been validated yet.
if not (ignore_errors or
isinstance(exc, (ValueError, TypeError))):
raise
else:
LOG.warning('Ignoring error adding implicit '