[checkmk-commits] 5283 FIX megaraid_ldisks: Fixed error during discovery: UnboundLocalError: local variable 'item' referenced before assignment

Module: check_mk Branch: master Commit: 915ef10ae6f2bf482c0034f6a16155391eedb213 URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=915ef10ae6f2bf… Author: Simon Betz <si(a)mathias-kettner.de> Date: Fri Sep 15 12:07:44 2017 +0200 5283 FIX megaraid_ldisks: Fixed error during discovery: UnboundLocalError: local variable 'item' referenced before assignment Change-Id: I7b5973901a587032aa19f513a6e2f49b807e7bd0 --- .werks/5283 | 11 +++++++++++ checks/megaraid_ldisks | 3 ++- 2 files changed, 13 insertions(+), 1 deletion(-) diff --git a/.werks/5283 b/.werks/5283 new file mode 100644 index 0000000..6de5102 --- /dev/null +++ b/.werks/5283 @@ -0,0 +1,11 @@ +Title: megaraid_ldisks: Fixed error during discovery: UnboundLocalError: local variable 'item' referenced before assignment +Level: 1 +Component: checks +Class: fix +Compatible: compat +Edition: cre +State: unknown +Version: 1.5.0i1 +Date: 1505469977 + + diff --git a/checks/megaraid_ldisks b/checks/megaraid_ldisks index 27f2ef3..797b791 100644 --- a/checks/megaraid_ldisks +++ b/checks/megaraid_ldisks @@ -42,6 +42,7 @@ def parse_megaraid_ldisks(info): parsed = {} adapter = None disk = None + item = None for line in info: l = " ".join(line) if line[0] == "Adapter" and not l.endswith('No Virtual Drive Configured.'): @@ -51,7 +52,7 @@ def parse_megaraid_ldisks(info): item = "%d/%d" % (adapter, disk) parsed[item] = {} - elif item in parsed.keys(): + elif item is not None and item in parsed.keys(): if line[0].startswith("State"): parsed[item]["state"] = l.split(":")[1].strip()