[checkmk-commits] Revert "5112 FIX if.include: Fixed wrong handling of duplicate interfaces if 'use alias' or ' use desccription' is configured"

Module: check_mk Branch: master Commit: a0ed7c25a7785027c443c73bdfd62edc9aecd33a URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=a0ed7c25a77850… Author: Simon Betz <si(a)mathias-kettner.de> Date: Tue Aug 22 15:11:45 2017 +0200 Revert "5112 FIX if.include: Fixed wrong handling of duplicate interfaces if 'use alias' or 'use desccription' is configured" This reverts commit 3254092ff6feefbd511c50c87d0dd3f0b6613651. --- .werks/5112 | 19 ------------------- checks/if.include | 2 +- 2 files changed, 1 insertion(+), 20 deletions(-) diff --git a/.werks/5112 b/.werks/5112 deleted file mode 100644 index 4c20abf..0000000 --- a/.werks/5112 +++ /dev/null @@ -1,19 +0,0 @@ -Title: if.include: Fixed wrong handling of duplicate interfaces if "use alias" or "use desccription" is configured -Level: 1 -Component: checks -Class: fix -Compatible: compat -Edition: cre -State: unknown -Version: 1.5.0i1 -Date: 1502973584 - -Within the rule {{Network Interface and Switch Port Discovery}} -you are able to configure the service description to use the -interface alias or description. Per default the index is used. -If there are duplicate interfaces with same name, the interface -index should be added. - - -Unfortunately, the afore-mentioned mechanism was broken. This -has been fixed. diff --git a/checks/if.include b/checks/if.include index a5d012c..d606a50 100644 --- a/checks/if.include +++ b/checks/if.include @@ -417,7 +417,7 @@ def inventory_if_common(info, has_nodeinfo = False): # Prepare single interfaces if not is_only_in_group: - if item in global_seen: + if ifIndex in global_seen: continue if item in seen_items: # duplicate