zuul/nodepool
Code Issues Proposed changes

Initialize retries attribute in SM launcher early

Browse Source 
Traceback (most recent call last):
  File "/opt/nodepool/lib/python3.10/site-packages/nodepool/driver/statemachine.py", line 741, in _runStateMachines
    sm.runStateMachine()
  File "/opt/nodepool/lib/python3.10/site-packages/nodepool/driver/statemachine.py", line 325, in runStateMachine
    self.attempts, self.retries, node.id)
AttributeError: 'StateMachineNodeLauncher' object has no attribute 'retries'

Change-Id: I92ce1ce7469c519b5d7e351781549aa98b5ab865
This commit is contained in:
Simon Westphahl
2023-06-13 12:07:49 +02:00
parent dfb498e797
commit 6ea5a4de9f
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
nodepool/driver/statemachine.py
View File

@@ -86,6 +86,7 @@ class StateMachineNodeLauncher(stats.StatsReporter):
self.manager = handler.manager
self.start_time = None
self.attempts = 0
self.retries = self.manager.provider.launch_retries
self.state_machine = None
# To handle deletions:
self.delete_state_machine = None
@@ -140,7 +141,6 @@ class StateMachineNodeLauncher(stats.StatsReporter):
self.keyscan_future = None
# Windows computer names can be no more than 15 chars long.
hostname = 'np' + self.node.id
self.retries = self.manager.provider.launch_retries
metadata = {'nodepool_node_id': self.node.id,
'nodepool_pool_name': self.handler.pool.name,
'nodepool_provider_name': self.manager.provider.name}