From c27327dfe01ba545f442788fb462ba0a98db0f45 Mon Sep 17 00:00:00 2001 From: Markus Hilger Date: Tue, 11 Aug 2026 05:17:59 +0200 Subject: [PATCH] Name the node when an inventory component is missing ConfluentTargetNotFound takes the node as its first argument, and every other caller passes it. The two inventory plugins constructed it with no arguments at all, so asking for a component that is not in the inventory map raised TypeError instead of returning the 404 the path was written to return. Both now follow the pattern used a few lines further down for volumes and name the component that was not found. --- confluent_server/confluent/plugins/hardwaremanagement/ipmi.py | 4 +++- .../confluent/plugins/hardwaremanagement/redfish.py | 4 +++- 2 files changed, 6 insertions(+), 2 deletions(-) diff --git a/confluent_server/confluent/plugins/hardwaremanagement/ipmi.py b/confluent_server/confluent/plugins/hardwaremanagement/ipmi.py index 85e29e4c..1e28a187 100644 --- a/confluent_server/confluent/plugins/hardwaremanagement/ipmi.py +++ b/confluent_server/confluent/plugins/hardwaremanagement/ipmi.py @@ -1053,7 +1053,9 @@ class IpmiHandler: await self.make_inventory_map() compname = self.invmap.get(component, None) if compname is None: - await self.output.put(msg.ConfluentTargetNotFound()) + await self.output.put(msg.ConfluentTargetNotFound( + self.node, + "No component named '{0}' found".format(component))) return invdata = await self.ipmicmd.get_inventory_of_component(compname) if invdata is None: diff --git a/confluent_server/confluent/plugins/hardwaremanagement/redfish.py b/confluent_server/confluent/plugins/hardwaremanagement/redfish.py index 86800d21..bcdf8bee 100644 --- a/confluent_server/confluent/plugins/hardwaremanagement/redfish.py +++ b/confluent_server/confluent/plugins/hardwaremanagement/redfish.py @@ -975,7 +975,9 @@ class IpmiHandler: await self.make_inventory_map() compname = self.invmap.get(component, None) if compname is None: - await self.output.put(msg.ConfluentTargetNotFound()) + await self.output.put(msg.ConfluentTargetNotFound( + self.node, + "No component named '{0}' found".format(component))) return invdata = await self.ipmicmd.get_inventory_of_component(compname) if invdata is None: