diff --git a/ironic_python_agent/hardware.py b/ironic_python_agent/hardware.py index 399d70609..54ceaddd0 100644 --- a/ironic_python_agent/hardware.py +++ b/ironic_python_agent/hardware.py @@ -498,7 +498,7 @@ def update_cached_node(): hardware_info=list_hardware_info(use_cache=True), timeout=API_LOOKUP_TIMEOUT, starting_interval=API_LOOKUP_INTERVAL, - uuid=uuid) + node_uuid=uuid) cache_node(content['node']) return content['node'] except Exception as exc: diff --git a/ironic_python_agent/tests/unit/test_hardware.py b/ironic_python_agent/tests/unit/test_hardware.py index 0ab04b673..5dc76b504 100644 --- a/ironic_python_agent/tests/unit/test_hardware.py +++ b/ironic_python_agent/tests/unit/test_hardware.py @@ -4533,7 +4533,7 @@ class TestAPIClientSaveAndUse(base.IronicAgentTest): hardware_info=mock.ANY, timeout=1, starting_interval=2, - uuid='node1') + node_uuid='node1') self.assertEqual(updated_node, hardware.NODE) calls = [mock.call('list_hardware_info'), mock.call('wait_for_disks')] diff --git a/releasenotes/notes/fix-auxillarly-node-lookup-argument-83d3f717c039e454.yaml b/releasenotes/notes/fix-auxillarly-node-lookup-argument-83d3f717c039e454.yaml new file mode 100644 index 000000000..7731d8f4d --- /dev/null +++ b/releasenotes/notes/fix-auxillarly-node-lookup-argument-83d3f717c039e454.yaml @@ -0,0 +1,9 @@ +--- +fixes: + - | + Fixes a minor incorrect keyword argument that was + matching between the method caller and the unit test but + not the actual method, unit test, and caller. This was + a non-fatal issue, and should now permit the agent to + attempt to lookup the node one last time before deploying + the instance image to pick-up a root device hint.