From 82dbaa96687848c5d5b95fe56c9ca635a023fef2 Mon Sep 17 00:00:00 2001 From: Julia Kreger Date: Mon, 14 Aug 2023 09:23:22 -0700 Subject: [PATCH] Log upon completion of power sync Previously, we updated node_periodic so we understood from the logs when a periodic task was completed, so we could understand where things were at in our hunt for database lock racess. In any event, we now explicitly log in the _sync_power_state method of the conductor, because it is not a node_periodic. Change-Id: Iaec9926fe031e65de4732ff0bc7988c5604d4755 --- ironic/conductor/manager.py | 3 +++ 1 file changed, 3 insertions(+) diff --git a/ironic/conductor/manager.py b/ironic/conductor/manager.py index 5efa3f66af..b27ef49d80 100644 --- a/ironic/conductor/manager.py +++ b/ironic/conductor/manager.py @@ -1440,6 +1440,9 @@ class ConductorManager(base_manager.BaseConductorManager): 'ConductorManager.PowerSyncNodesCount', len(nodes)) + LOG.debug('Completed power state sync operation, evaluated %s ' + 'nodes.', len(futures)) + def _sync_power_state_nodes_task(self, context, nodes): """Invokes power state sync on nodes from synchronized queue.