diff options
author | Zuul <zuul@review.opendev.org> | 2023-02-15 19:15:40 +0000 |
---|---|---|
committer | Gerrit Code Review <review@openstack.org> | 2023-02-15 19:15:40 +0000 |
commit | 124fc46f9c2b05ee611ee50a78b7c6522d97cf5e (patch) | |
tree | 797ee7728c4031229f70707bd296b4b164d15db9 | |
parent | 776be1e7ec035679cc075d664fb6a0af5daea9d8 (diff) | |
parent | 1897ac4c6ff6669e569022bb9cab7f04575cec11 (diff) | |
download | zuul-124fc46f9c2b05ee611ee50a78b7c6522d97cf5e.tar.gz |
Merge "Prevent files cache ltime from going backward"
-rw-r--r-- | zuul/configloader.py | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/zuul/configloader.py b/zuul/configloader.py index f01be597b..f9e52595e 100644 --- a/zuul/configloader.py +++ b/zuul/configloader.py @@ -2259,6 +2259,12 @@ class TenantParser(object): job.source_context.branch) with self.unparsed_config_cache.writeLock( 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 # for ALL tenants the project is a part of, we can # clear the whole cache and then populate it with the |