openstack/ironic
Code Issues Proposed changes

Log how long power sync and sensor collections take

Browse Source 
Change-Id: I1632d6e6dd87dd6dc71b0fd86d8b9cd06279d7dc
Signed-off-by: Dmitry Tantsur <dtantsur@protonmail.com>
This commit is contained in:
Dmitry Tantsur
2025-07-24 11:32:06 +02:00
parent 60e10d1266
commit ecf1571921
1 changed files with 8 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
ironic/conductor/manager.py
View File

@@ -1538,6 +1538,7 @@ class ConductorManager(base_manager.BaseConductorManager):
def _sync_power_states(self, context):
"""Periodic task to sync power states for the nodes."""
filters = {'maintenance': False}
started = time.time()
# NOTE(etingof): prioritize non-responding nodes to fail them fast
nodes = sorted(
@@ -1578,8 +1579,9 @@ class ConductorManager(base_manager.BaseConductorManager):
'ConductorManager.PowerSyncNodesCount',
len(nodes))
LOG.debug('Completed power state sync operation, evaluated %s '
'nodes.', len(futures))
LOG.debug('Completed power state sync operation, evaluated %d '
'nodes with %d workers in %.2f seconds',
len(futures), number_of_workers, time.time() - started)
def _sync_power_state_nodes_task(self, context, nodes):
"""Invokes power state sync on nodes from synchronized queue.
@@ -2878,6 +2880,7 @@ class ConductorManager(base_manager.BaseConductorManager):
if not CONF.sensor_data.enable_for_undeployed_nodes:
filters['provision_state'] = states.ACTIVE
started = time.time()
nodes = queue.Queue()
for node_info in self.iter_nodes(fields=['instance_uuid'],
filters=filters):
@@ -2903,6 +2906,9 @@ class ConductorManager(base_manager.BaseConductorManager):
if not_done:
LOG.warning("%d workers for send sensors data did not complete",
len(not_done))
LOG.debug('Completed sending sensor data, evaluated %d '
'nodes with %d workers in %.2f seconds',
len(done), number_of_threads, time.time() - started)
def _filter_out_unsupported_types(self, sensors_data):
"""Filters out sensor data types that aren't specified in the config.