openstack/ironic
Code Issues Proposed changes

Fix log statement about starting inspection

Browse Source 
Change-Id: I3e5a63100882774e85235f6f0d8aeb0c2a9610ca
This commit is contained in:
Mahnoor Asghar 2024-06-27 10:16:10 -04:00
parent 25b2564bde
commit e95d5a8446
2 changed files with 6 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
ironic/drivers/modules/inspector/agent.py
View File

@ -45,7 +45,7 @@ class AgentInspect(common.Common):
self.hooks = hooks_base.validate_inspection_hooks()
def _start_managed_inspection(self, task):
"""Start inspection managed by ironic."""
"""Start inspection with boot managed by ironic."""
ep = deploy_utils.get_ironic_api_url().rstrip('/')
if ep.endswith('/v1'):
ep = f'{ep}/continue_inspection'

11
ironic/drivers/modules/inspector/interface.py
View File

@ -190,8 +190,7 @@ class Common(base.InspectInterface):
def inspect_hardware(self, task):
"""Inspect hardware to obtain the hardware properties.
This particular implementation only starts inspection using
ironic-inspector. Results will be checked in a periodic task.
Results will be checked in a periodic task.
:param task: a task from TaskManager.
:returns: states.INSPECTWAIT
@ -214,8 +213,8 @@ class Common(base.InspectInterface):
inspect_utils.cache_lookup_addresses(task.node)
task.node.save()
LOG.debug('Starting inspection for node %(uuid)s using '
'ironic-inspector, booting is managed by %(project)s',
LOG.debug('Starting inspection for node %(uuid)s. Booting is '
'managed by %(project)s',
{'uuid': task.node.uuid,
'project': 'ironic' if manage_boot else 'ironic-inspector'})
@ -237,7 +236,7 @@ class Inspector(Common):
"""In-band inspection via ironic-inspector project."""
def _start_managed_inspection(self, task):
"""Start inspection managed by ironic."""
"""Start inspection with boot managed by ironic."""
cli = client.get_client(task.context)
endpoint = _get_callback_endpoint(cli)
prepare_managed_inspection(task, endpoint)
@ -313,7 +312,7 @@ def _start_inspection(node_uuid, context):
def _check_status(task):
"""Check inspection status for node given by a task."""
"""Check inspection status from inspector for node given by a task."""
node = task.node
if node.provision_state != states.INSPECTWAIT:
return