From a748cc3032298180421f904f410106f5d3ec2f3a Mon Sep 17 00:00:00 2001 From: Jarrod Johnson Date: Tue, 5 Nov 2019 13:28:12 -0500 Subject: [PATCH] Remove incorrect check for HOSTUNREACH This seems incorrect and further redundant with other portions of the code. This was causing a 'has no len()' error when encountered. --- .../confluent/plugins/hardwaremanagement/redfish.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/confluent_server/confluent/plugins/hardwaremanagement/redfish.py b/confluent_server/confluent/plugins/hardwaremanagement/redfish.py index cb62de95..5f8e3bc5 100644 --- a/confluent_server/confluent/plugins/hardwaremanagement/redfish.py +++ b/confluent_server/confluent/plugins/hardwaremanagement/redfish.py @@ -175,7 +175,7 @@ class IpmiCommandWrapper(ipmicommand.Command): raise exc.TargetEndpointUnreachable(se.strerror) else: raise exc.TargetEndpointUnreachable(str(se)) - if isinstance(se, socket.timeout) or (len(se) > 1 and se[1] == 'EHOSTUNREACH'): + if isinstance(se, socket.timeout): raise exc.TargetEndpointUnreachable('timeout') raise except pygexc.PyghmiException as pe: