zuul/nodepool
Code Issues Proposed changes

Merge "Delete init nodes when resetting lost requests"

Browse Source
This commit is contained in:
Zuul 2022-04-28 06:04:39 +00:00 committed by Gerrit Code Review
commit 1fb3c69fa1
1 changed files with 8 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
nodepool/launcher.py
View File

@ -486,6 +486,14 @@ class CleanupWorker(BaseCleanupWorker):
"request", node.id)
return
# If the node is in state init then the launcher that worked
# on the lost request has been interrupted between creating
# the znode and locking/setting to building. In this case the
# znode is leaked and we should delete the node instead of
# just deallocating it.
if node.state == zk.INIT:
node.state = zk.DELETING
node.allocated_to = None
try:
zk_conn.storeNode(node)