zuul/zuul
Code Issues Proposed changes

Remove unecessary code from manager._postConfig

Browse Source 
We don't need to clear the layout cache in _postConfig because
the configloader creates a new pipeline manager in all cases
where this method would be called anyway (that is: a tenant
reconfiguration).  Remove this unecessary code and the potentially
confusing comment that goes with it.

Change-Id: I7ee0f08688a084d6fd706877f1453f8fd29a9e65
This commit is contained in:
James E. Blair
2025-03-31 11:13:59 -07:00
parent b72800266c
commit 3d97866e0d
1 changed files with 0 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
zuul/manager/__init__.py
View File

@@ -106,12 +106,6 @@ class PipelineManager(metaclass=ABCMeta):
self.current_context = None
def _postConfig(self):
# After a reconfiguration, we need to forget anything built
# from the old layout.
# TODO: we may be able to preserve change queues if projects
# haven't changed their pipeline participation.
self._layout_cache = {}
layout = self.tenant.layout
self.buildChangeQueues(layout)
# Make sure we have state and change list objects. We