[checkmk-commits] [tribe29/checkmk] 35ea19: Rename FetcherResult -> FetcherMessage

Branch: refs/heads/master Home: https://github.com/tribe29/checkmk Commit: 35ea199111337e7bb1b4cd6ab72474a07542ef5f https://github.com/tribe29/checkmk/commit/35ea199111337e7bb1b4cd6ab72474a07… Author: Mathias Laurin <mathias.laurin(a)tribe29.com> Date: 2020-09-11 (Fri, 11 Sep 2020) Changed paths: M cmk/base/checking.py M cmk/base/data_sources/_data_sources.py M cmk/base/discovery.py M cmk/fetchers/type_defs.py Log Message: ----------- Rename FetcherResult -> FetcherMessage The name `Result` is already heavily overloaded and it should take a new meaning with the error handling. Currently, the FetcherResult/FetcherMessage type contains a piece of FetcherHeader and the fetcher payload in an untyped dict. To proper handle errors, we need the full FetcherHeader. It is not really good practice anyway to tear objects appart. This will be fixed in a comming commit. This is an automated, mechanical change using `ack | xargs sed` CMK-5419 Change-Id: I594c5be777cc45f702e613bb6241732c1df67f24 Commit: 83c7211857a4e1715217da3cfea3bf16dff7fd76 https://github.com/tribe29/checkmk/commit/83c7211857a4e1715217da3cfea3bf16d… Author: Sven Panne <sven.panne(a)tribe29.com> Date: 2020-09-11 (Fri, 11 Sep 2020) Changed paths: M livestatus/src/TableHostsByGroup.cc M livestatus/src/TableServicesByGroup.cc M livestatus/src/TableServicesByHostGroup.cc Log Message: ----------- No need to copy whole hosts/services around just for grouping. Improved const-correctness a bit on the way. Change-Id: I06d3bfadd592a8202087639062e489238c77b126 Compare: https://github.com/tribe29/checkmk/compare/256e347cb8f6...83c7211857a4