From 8ac115a8a77f6ff14acb8faa55f30dcf0f6d0545 Mon Sep 17 00:00:00 2001 From: Jarrod Johnson Date: Wed, 20 Sep 2023 15:20:49 -0400 Subject: [PATCH] Restore sane timeout behavior to pyghmi The change to make sure resources are freed replaced straightforward message with a weird one. In the well known context, put the well known message back in the reasonable path. Change-Id: Iea17b8bbb7b14255debb05a90d5701f320b5d705 --- pyghmi/ipmi/private/session.py | 3 +++ 1 file changed, 3 insertions(+) diff --git a/pyghmi/ipmi/private/session.py b/pyghmi/ipmi/private/session.py index 70ec0e45..1c99a96d 100644 --- a/pyghmi/ipmi/private/session.py +++ b/pyghmi/ipmi/private/session.py @@ -592,6 +592,9 @@ class Session(object): def onlogon(self, parameter): if 'error' in parameter: + while self.logonwaiters: + waiter = self.logonwaiters.pop() + waiter(parameter) self._mark_broken(parameter['error']) elif self.onlogpayload: self._cmdwait()