[checkmk-commits] 4804 FIX brocade_fcport: Fix limiting discovered ports by operational and physical status

Module: check_mk Branch: master Commit: 9e6589c52a39c76bb8c35fcd70a4bc731fe792bf URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=9e6589c52a39c7… Author: Konstantin Büttner <kb(a)mathias-kettner.de> Date: Mon Aug 14 17:19:28 2017 +0200 4804 FIX brocade_fcport: Fix limiting discovered ports by operational and physical status Previously, any rule set aiming to influence this would simply have no effect, as the defaults were always used. This has been fixed. Change-Id: I0348fc56cf5aaf78b152bccff7134417edb21ac2 --- .werks/4804 | 12 ++++++++++++ checks/brocade_fcport | 4 ++-- 2 files changed, 14 insertions(+), 2 deletions(-) diff --git a/.werks/4804 b/.werks/4804 new file mode 100644 index 0000000..bad9a08 --- /dev/null +++ b/.werks/4804 @@ -0,0 +1,12 @@ +Title: brocade_fcport: Fix limiting discovered ports by operational and physical status +Level: 1 +Component: checks +Class: fix +Compatible: compat +Edition: cre +State: unknown +Version: 1.5.0i1 +Date: 1502723516 + +Previously, any rule set aiming to influence this would simply have no effect, +as the defaults were always used. This has been fixed. diff --git a/checks/brocade_fcport b/checks/brocade_fcport index a9a9a9f..0b115ed 100644 --- a/checks/brocade_fcport +++ b/checks/brocade_fcport @@ -214,8 +214,8 @@ def inventory_brocade_fcport(parsed): opstate = if_entry["opstate"] admstate = if_entry["admstate"] if admstate in admstates and \ - opstate in brocade_fcport_inventory_opstates and \ - phystate in brocade_fcport_inventory_phystates: + opstate in opstates and \ + phystate in phystates: inventory.append((brocade_fcport_getitem(parsed, if_entry["index"], if_entry["portname"], if_entry["is_isl"], uses_portname, shows_isl), '{ "phystate": [%d], "opstate": [%d], "admstate": [%d] }' % \