summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorZuul <zuul@review.opendev.org>2020-05-05 23:04:23 +0000
committerGerrit Code Review <review@openstack.org>2020-05-05 23:04:23 +0000
commit4f31fd67b062e4ebf36500b968079c86e293661e (patch)
treebcafccf081625c1026fcce2e2a6428c3f4ec748d
parentcdb2872acb62de3fb289a792db08dc39602e4bfb (diff)
parent3c44f8e9ff4ce5cbbf7ac389c79ce091b5fff1a9 (diff)
downloadzuul-4f31fd67b062e4ebf36500b968079c86e293661e.tar.gz
Merge "merger: warn about invalid object type"
-rw-r--r--zuul/merger/merger.py4
1 files changed, 4 insertions, 0 deletions
diff --git a/zuul/merger/merger.py b/zuul/merger/merger.py
index fcaf6e81f..7e1188730 100644
--- a/zuul/merger/merger.py
+++ b/zuul/merger/merger.py
@@ -556,6 +556,7 @@ class Repo(object):
def getFiles(self, files, dirs=[], branch=None, commit=None,
zuul_event_id=None):
+ log = get_annotated_logger(self.log, zuul_event_id)
ret = {}
repo = self.createRepoObject(zuul_event_id)
if branch:
@@ -564,6 +565,9 @@ class Repo(object):
tree = repo.commit(commit).tree
for fn in files:
if fn in tree:
+ if tree[fn].type != 'blob':
+ log.warning(
+ "%s: object %s is not a blob", self.local_path, fn)
ret[fn] = tree[fn].data_stream.read().decode('utf8')
else:
ret[fn] = None