[checkmk-commits] [tribe29/checkmk] 83440a: Error handling: check & summarize

Branch: refs/heads/master Home: https://github.com/tribe29/checkmk Commit: 83440a06fd77ab994fbd6eeadfe5ccbc2d4776cb https://github.com/tribe29/checkmk/commit/83440a06fd77ab994fbd6eeadfe5ccbc2… Author: Mathias Laurin <mathias.laurin(a)tribe29.com> Date: 2020-09-15 (Tue, 15 Sep 2020) Changed paths: M cmk/base/automations/check_mk.py M cmk/base/checking.py M cmk/base/data_sources/_abstract.py M cmk/base/data_sources/_data_sources.py M cmk/base/data_sources/agent.py M cmk/base/data_sources/snmp.py M cmk/base/discovery.py M cmk/base/inventory.py M cmk/base/modes/check_mk.py M tests/unit/cmk/base/data_sources/test_agent.py M tests/unit/cmk/base/data_sources/test_cmd_caching.py M tests/unit/cmk/base/data_sources/test_host_sections.py M tests/unit/cmk/base/data_sources/test_snmp_data_source.py M tests/unit/cmk/base/data_sources/test_tcp_data_source.py Log Message: ----------- Error handling: check & summarize We wrap the argument to `check()` (aka `parse()` + `summarize()`) into our `Result` type in order to encapsulate the error handling at the appropriate level. `fetch()` is missing as Result there needs to be serialized. Furthermore, starting with `check()` lets us fix the API and remove parts of the unnecessary complexity in the next commits. CMK-5419 Change-Id: Iceca2a231b9cfc070083ce084ebb5b991e1bca1b