[checkmk-commits] Check_MK Git: check_mk: #1236 FIX multipath: Now show correct error in case of removed multipaths instead of check crash

Module: check_mk Branch: master Commit: 7bdf6bd07fdd878fc13c2e9c947cf7201548d0bc URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=7bdf6bd07fdd87… Author: Bastian Kuhn <bk(a)mathias-kettner.de> Date: Thu Mar 19 14:33:35 2015 +0100 #1236 FIX multipath: Now show correct error in case of removed multipaths instead of check crash --- .werks/1236 | 10 ++++++++++ ChangeLog | 1 + checks/multipath | 2 +- 3 files changed, 12 insertions(+), 1 deletion(-) diff --git a/.werks/1236 b/.werks/1236 new file mode 100644 index 0000000..c2c2510 --- /dev/null +++ b/.werks/1236 @@ -0,0 +1,10 @@ +Title: multipath: Now show correct error in case of removed multipaths instead of check crash +Level: 1 +Component: checks +Class: fix +Compatible: compat +State: unknown +Version: 1.2.7i1 +Date: 1426771976 + + diff --git a/ChangeLog b/ChangeLog index 7f22dfc..37858b8 100644 --- a/ChangeLog +++ b/ChangeLog @@ -265,6 +265,7 @@ * 2058 FIX: ucs_bladecenter_fans.temp, ucs_bladecenter_psu.chassis_temp: fixed broken temperature checks (nagios only)... * 2059 FIX: ucs_bladecenter_if: fixed exception when fibrechannel interfaces were not configured... * 1233 FIX: Fixed fileinfo check for solaris in case of missing files + * 1236 FIX: multipath: Now show correct error in case of removed multipaths instead of check crash Multisite: * 1758 Improved exception hander: Shows details without additional debug request, added mailto link for error report... diff --git a/checks/multipath b/checks/multipath index b212b83..64dda25 100644 --- a/checks/multipath +++ b/checks/multipath @@ -289,7 +289,7 @@ def check_multipath(item, target_numpaths, parsed): mmap = parsed[item] else: for mmap in parsed.values(): - if mmap["alias"] == item: + if mmap.get("alias") == item: break else: return 3, "Multipath device not found in agent output"