[checkmk-commits] [tribe29/checkmk] 1422fe: Various checker cleanups

Branch: refs/heads/master Home: https://github.com/tribe29/checkmk Commit: 1422fe152105c7df9b8e6b4c4562643f75185a04 https://github.com/tribe29/checkmk/commit/1422fe152105c7df9b8e6b4c4562643f7… Author: Lars Michelsen <lm(a)tribe29.com> Date: 2020-10-07 (Wed, 07 Oct 2020) Changed paths: M cmk/base/checkers/_checkers.py M cmk/base/checkers/snmp.py M cmk/base/inventory.py Log Message: ----------- Various checker cleanups * Add more verbose output to make it easier to understand processing * Fix variable name host_section to source_result * Fix wrong type of SNMPSource.prefetched_sections CMK-5758 Change-Id: I85d633396c24ea5e1a8c33fc055827b33305a2e8 Commit: fb99796a004a9cc5f45a8aef5ad6752d599fe33d https://github.com/tribe29/checkmk/commit/fb99796a004a9cc5f45a8aef5ad6752d5… Author: Lars Michelsen <lm(a)tribe29.com> Date: 2020-10-07 (Wed, 07 Oct 2020) Changed paths: M cmk/base/checkers/snmp.py M cmk/base/inventory.py Log Message: ----------- Drop SNMP communication from checkers caused by structured status inventory This case (SNMP source + multi_host_sections given) was only reached in the checker processes. In these processes we must not perform any fetching. The needed data is already fetched by the fetcher anyway, so we can simply remove the logic here. CMK-5758 Change-Id: Ifb01d9ba3d8b36c7b6990473fc4f877d0ccb06fb Commit: 8081c16b385c98eeed4487c73a4984c8afa0d71a https://github.com/tribe29/checkmk/commit/8081c16b385c98eeed4487c73a4984c8a… Author: Lars Michelsen <lm(a)tribe29.com> Date: 2020-10-07 (Wed, 07 Oct 2020) Changed paths: M cmk/base/inventory.py Log Message: ----------- Inventory: Extract multi host section gathering Previously _do_inv_for_realhost() was fetching the multi_host_sections on demand in case they were not provided by the call site. This commit completely removes that case from the functions _do_inv_for and _do_inv_for_realhost. The multi host section computation is now done in advance by all call sites. The MHS are either built based on the data fetched from the fetcher helper processes or fetched ad-hoc by calls that need to fetch the data on their own (cmk -i and inventory check). Change-Id: I608c53b8dcfe7d8c567570d3bfcf380c52437cba Compare: https://github.com/tribe29/checkmk/compare/ddcc6530ea95...8081c16b385c