[checkmk-commits] [tribe29/checkmk] 313c2e: Add tests for cisco_sensor_item.include

Branch: refs/heads/1.5.0 Home: https://github.com/tribe29/checkmk Commit: 313c2e144abc8a5935a273adeaf597d239736ac7 https://github.com/tribe29/checkmk/commit/313c2e144abc8a5935a273adeaf597d23… Author: Tom Baerwinkel <tom.baerwinkel(a)tribe29.com> Date: 2019-11-26 (Tue, 26 Nov 2019) Changed paths: A tests/unit/checks/test_cisco_sensor_item_include.py Log Message: ----------- Add tests for cisco_sensor_item.include SUP-1192 Change-Id: Iec02736b0918f8a0ce280f075ff3728aa6c6983b Commit: 65e8f0f0e343959530ef1c73a842dd03d9185232 https://github.com/tribe29/checkmk/commit/65e8f0f0e343959530ef1c73a842dd03d… Author: Tom Baerwinkel <tom.baerwinkel(a)tribe29.com> Date: 2019-11-26 (Tue, 26 Nov 2019) Changed paths: M checks/cisco_sensor_item.include M checks/cisco_temp_perf Log Message: ----------- Minor cleanup for cisco_sensor_item * rename trial_string to description * rename fall_back to sensor_id since it's numeric * remove unnecessary lstrip() * remove comments that explain the control flow * remove commented out code mentioning cisco_sensor_item SUP-1192 Change-Id: I6072a69992cb3857d6add79bb9abb0555cb8dac9 Commit: e5cee87492f97964213270f2119b6303c4183bf2 https://github.com/tribe29/checkmk/commit/e5cee87492f97964213270f2119b6303c… Author: Tom Baerwinkel <tom.baerwinkel(a)tribe29.com> Date: 2019-11-26 (Tue, 26 Nov 2019) Changed paths: A .werks/10463 M checks/cisco_sensor_item.include M tests/unit/checks/test_cisco_sensor_item_include.py Log Message: ----------- 10463 FIX cisco_power: discover unique items in one edge case For some sensor descriptions the cisco_power check discovered items that were not unique. One description was e.g. "Sw1, PSA Normal" which was translated into the item "Sw1". Another sensor that used the description "Sw1, PSB Normal" was then aliased to "Sw1" as well. Such items can now be differentiated. The example sensors would now be mapped to the items "Sw1 PSA {ID}" and "Sw1 PSB {ID}" where {ID} is the ID of the sensor. The affected services have to be rediscovered. SUP-1192 Change-Id: I3895b355595ae69171daa9eae003f98e5c0b783c Compare: https://github.com/tribe29/checkmk/compare/07c88d5a3440...e5cee87492f9