zuul/zuul
Code Issues Proposed changes

Merge "Update build/job versions in place"

Browse Source
This commit is contained in:
Zuul
2023-02-11 20:48:00 +00:00
committed by Gerrit Code Review
parent fbddc72ec8 6a4f752ca6
commit c6b98116f5
1 changed files with 4 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
zuul/model.py
View File

@@ -4449,9 +4449,8 @@ class BuildSet(zkobject.ZKObject):
version = build.getZKVersion()
# If zstat is None, we created the object
if version is not None:
versions = self.build_versions.copy()
versions[build.uuid] = version + 1
self.updateAttributes(context, build_versions=versions)
self.build_versions[build.uuid] = version + 1
self.updateAttributes(context, build_versions=self.build_versions)
def updateJobVersion(self, context, job):
if (COMPONENT_REGISTRY.model_api < 12):
@@ -4459,9 +4458,8 @@ class BuildSet(zkobject.ZKObject):
version = job.getZKVersion()
if version is not None:
versions = self.job_versions.copy()
versions[job.name] = version + 1
self.updateAttributes(context, job_versions=versions)
self.job_versions[job.name] = version + 1
self.updateAttributes(context, job_versions=self.job_versions)
def shouldRefreshBuild(self, build):
# Unless all schedulers are updating versions, we can't trust