zuul/zuul
Code Issues Proposed changes

Merge "Prevent files cache ltime from going backward"

Browse Source
This commit is contained in:
Zuul 2023-02-15 19:15:40 +00:00 committed by Gerrit Code Review
commit 124fc46f9c
1 changed files with 6 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
zuul/configloader.py
View File

@ -2259,6 +2259,12 @@ class TenantParser(object):
job.source_context.branch) job.source_context.branch)
with self.unparsed_config_cache.writeLock( with self.unparsed_config_cache.writeLock(
job.source_context.project_canonical_name): job.source_context.project_canonical_name):
# Prevent files cache ltime from going backward
if files_cache.ltime >= job.ltime:
self.log.info(
"Discarding job %s result since the files cache was "
"updated in the meantime", job)
continue
# Since the cat job returns all required config files # Since the cat job returns all required config files
# for ALL tenants the project is a part of, we can # for ALL tenants the project is a part of, we can
# clear the whole cache and then populate it with the # clear the whole cache and then populate it with the