From aa7701ea3c5a8b2015faa5bb46bc65f99a3690c3 Mon Sep 17 00:00:00 2001 From: Jarrod Johnson Date: Tue, 11 May 2021 08:02:19 -0400 Subject: [PATCH] More usefully indicate unfound interfaces Rather than no data, provide not found error when requesting a currently unsupported name. --- confluent_server/confluent/plugins/hardwaremanagement/ipmi.py | 4 ++++ .../confluent/plugins/hardwaremanagement/redfish.py | 4 ++++ 2 files changed, 8 insertions(+) diff --git a/confluent_server/confluent/plugins/hardwaremanagement/ipmi.py b/confluent_server/confluent/plugins/hardwaremanagement/ipmi.py index 3a1706b2..402293f0 100644 --- a/confluent_server/confluent/plugins/hardwaremanagement/ipmi.py +++ b/confluent_server/confluent/plugins/hardwaremanagement/ipmi.py @@ -749,6 +749,10 @@ class IpmiHandler(object): self.node, 'Invalid prefix length given')) else: raise + elif len(self.element) == 4 and self.element[-1] != 'management': + self.output.put( + msg.ConfluentTargetNotFound(self.node, + 'Interface not found')) def handle_users(self): # Create user diff --git a/confluent_server/confluent/plugins/hardwaremanagement/redfish.py b/confluent_server/confluent/plugins/hardwaremanagement/redfish.py index 53cddfc5..09ec17e1 100644 --- a/confluent_server/confluent/plugins/hardwaremanagement/redfish.py +++ b/confluent_server/confluent/plugins/hardwaremanagement/redfish.py @@ -608,6 +608,10 @@ class IpmiHandler(object): self.node, 'Invalid prefix length given')) else: raise + elif len(self.element) == 4 and self.element[-1] != 'management': + self.output.put( + msg.ConfluentTargetNotFound(self.node, + 'Interface not found')) def handle_users(self): # Create user