[checkmk-commits] #2854 FIX winperf_if: Assuming interfaces without status to be up

Module: check_mk Branch: master Commit: 918e40c259a13a121bb0d2a2b27bad066b6ece19 URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=918e40c259a13a… Author: Lars Michelsen <lm(a)mathias-kettner.de> Date: Wed Dec 2 13:30:37 2015 +0100 #2854 FIX winperf_if: Assuming interfaces without status to be up --- .werks/2854 | 10 ++++++++++ ChangeLog | 1 + checks/winperf_if | 4 +++- 3 files changed, 14 insertions(+), 1 deletion(-) diff --git a/.werks/2854 b/.werks/2854 new file mode 100644 index 0000000..d3cc492 --- /dev/null +++ b/.werks/2854 @@ -0,0 +1,10 @@ +Title: winperf_if: Assuming interfaces without status to be up +Level: 1 +Component: checks +Class: fix +Compatible: compat +State: unknown +Version: 1.2.7i4 +Date: 1449059417 + + diff --git a/ChangeLog b/ChangeLog index 654398b..a84ad8e 100644 --- a/ChangeLog +++ b/ChangeLog @@ -120,6 +120,7 @@ * 2873 FIX: megaraid_pdisks: handle case "Copyback" correctly... * 2852 FIX: check_http: SSL can now be configured to use specific SSL version * 2853 FIX: cisco_asa_failover: Added support for CISCO PIX security devices + * 2854 FIX: winperf_if: Assuming interfaces without status to be up Multisite: * 2684 Added icons for downloading agent data / walks of hosts... diff --git a/checks/winperf_if b/checks/winperf_if index 9843a5f..99fd6e2 100644 --- a/checks/winperf_if +++ b/checks/winperf_if @@ -220,7 +220,9 @@ def convert_winperf_if(info, nic_names, nic_index): # NetConnectionTable Stuff: # # if we have no status, but link information, we assume IF is connected - connection_status = nic.get('NetConnectionStatus', '2') + connection_status = nic.get('NetConnectionStatus') + if not connection_status: + connection_status = '2' # Windows NetConnectionStatus Table connection_states = {