commit ff8035f8482a1ee9f8ef84ef9ee16ab3f2ba9cd2
author Obihörnchen <Markus.Hilger@de.ibm.com> Wed Dec 05 21:07:37 2018 +0100
committer Andrew Geissler <geissonator@yahoo.com> Mon Dec 10 08:03:09 2018 -0600
tree 84dec1d73ad4664d003c98225e2b3af757512a84
parent 9f4b45656843ebaaa922368d20d9f33484755693

Fix TypeError in fru status vebose output

It is a list, not a dictionary.
Fixes: `TypeError: list indices must be integers, not unicode`

Change-Id: Ieff2d833867255a8019129b4b69f0353e3fc967e
Signed-off-by: Markus Hilger <Markus.Hilger@de.ibm.com>
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>

thalerj/openbmctool.py

diff --git a/thalerj/openbmctool.py b/thalerj/openbmctool.py
index 4dc7462..9905cb0 100755
--- a/thalerj/openbmctool.py
+++ b/thalerj/openbmctool.py
@@ -467,8 +467,8 @@
                 if hasSels:
                     loglist = []
                     faults = frulist[key+"/fault"]['endpoints']
-                    for key in faults:
-                        loglist.append(faults[key].split('/')[-1])
+                    for item in faults:
+                        loglist.append(item.split('/')[-1])
                     frus[fruName] = {"compName": fruName, "Functional": "No", "Present":boolToString(present), "IsFru": boolToString(isFru), "selList": ', '.join(loglist).strip() }
                 else:
                     frus[fruName] = {"compName": fruName, "Functional": "Yes", "Present":boolToString(present), "IsFru": boolToString(isFru), "selList": "None" }