[checkmk-commits] [tribe29/checkmk] ef6ff8: 10267 FIX dell_compellent_disks would not see more...

Branch: refs/heads/1.5.0 Home: https://github.com/tribe29/checkmk Commit: ef6ff835d0482fdbaf2500612905007f94f77410 https://github.com/tribe29/checkmk/commit/ef6ff835d0482fdbaf2500612905007f9… Author: Mathias Laurin <mathias.laurin(a)tribe29.com> Date: 2019-09-30 (Mon, 30 Sep 2019) Changed paths: A .werks/10267 M checks/dell_compellent_disks M tests/unit/checks/generictests/datasets/dell_compellent_controller_regression.py M tests/unit/checks/generictests/datasets/dell_compellent_disks_regression.py M tests/unit/checks/generictests/datasets/dell_compellent_enclosure_regression.py Log Message: ----------- 10267 FIX dell_compellent_disks would not see more than 9 disks The dell_compellent_disks check was using the the inventory function from `dell_compellent.include`. However, this inventory function expects a list where `parse_dell_compellent_disks` builds a dict. This resulted in the inventory function returning the first letter of every key from the parse function, that is, disk #7, would be correctly inventoried as #7 but disk #42 would be inventoried as #4. The inventory function would also never see more than 9 disks with no guarantee that it would always see the same 9 disks. This patch fixes the inventory function to dell_compellent_disks and test that the error is not present in the other Compellent checks. This fixes FEED-4283. Change-Id: I16cfd52f220b7cc2625443ed8bf4f1ee2a748451