From 35b50fb705f11eadf71b35c5065c220fdbf83e8b Mon Sep 17 00:00:00 2001 From: Jarrod Johnson Date: Sat, 14 Sep 2013 10:03:56 -0400 Subject: [PATCH] Fix pyghmi behavior on BMC NACK When BMC sent NACK, pyghmi incorrectly attempted to construct new payload by concatenating a tuple and str. Fix by converting the tuple to str prior to sending. Change-Id: Ia07384651870bf01e7dc24757e1e00018ab84556 --- pyghmi/ipmi/console.py | 1 + 1 file changed, 1 insertion(+) diff --git a/pyghmi/ipmi/console.py b/pyghmi/ipmi/console.py index 4669a5c0..2399a498 100644 --- a/pyghmi/ipmi/console.py +++ b/pyghmi/ipmi/console.py @@ -245,6 +245,7 @@ class Console(object): else: # retry all or part of packet, but in a new form # also add pending output for efficiency and ease newtext = self.lastpayload[4 + ackcount:] + newtext = struct.pack("B"*len(newtext), *newtext) self.pendingoutput = newtext + self.pendingoutput self._sendpendingoutput() elif self.awaitingack: # session marked us as happy, but we are not