From a6779e93e140b2a115d2bdd55513e4a1bc2c773b Mon Sep 17 00:00:00 2001 From: Jarrod Johnson Date: Thu, 5 May 2022 12:26:12 -0400 Subject: [PATCH] More aggressively give up on initial connection Previously, logon retries were not part of the normal tries and treated separately. Now that the normal retry logic is in place, a change is needed to avoid the wait times amplifying. Change-Id: Ia2e5b4f0790ce4574f17cf7ceb15ebd7a36f8afa --- pyghmi/ipmi/private/session.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/pyghmi/ipmi/private/session.py b/pyghmi/ipmi/private/session.py index bd3bee93..c9cfd0d8 100644 --- a/pyghmi/ipmi/private/session.py +++ b/pyghmi/ipmi/private/session.py @@ -507,7 +507,7 @@ class Session(object): self.onlogpayloadtype = None self.logoutexpiry = None self.autokeepalive = keepalive - self.maxtimeout = 3 # be aggressive about giving up on initial packet + self.maxtimeout = 2 # be aggressive about giving up on initial packet self.incommand = False self.nameonly = 16 # default to name only lookups in RAKP exchange self.servermode = False @@ -1126,7 +1126,7 @@ class Session(object): data=[0x8e, self.privlevel]) def login(self): - self.logontries = 5 + self.logontries = 2 self._initsession() self._get_channel_auth_cap()