[checkmk-commits] [tribe29/checkmk] 732292: Source success is independent of content of payload

Branch: refs/heads/master Home: https://github.com/tribe29/checkmk Commit: 732292927e19198b06bdfa8567172833e117b49e https://github.com/tribe29/checkmk/commit/732292927e19198b06bdfa8567172833e… Author: Moritz Kiemer <moritz.kiemer(a)tribe29.com> Date: 2022-03-23 (Wed, 23 Mar 2022) Changed paths: M cmk/base/agent_based/checking/__init__.py M cmk/base/agent_based/discovery/__init__.py M cmk/base/agent_based/inventory/__init__.py M cmk/base/agent_based/utils.py M cmk/base/automations/check_mk.py M cmk/base/modes/check_mk.py M cmk/base/sources/_abstract.py M cmk/core_helpers/__init__.py M cmk/core_helpers/_base.py M cmk/core_helpers/agent.py M cmk/core_helpers/ipmi.py M cmk/core_helpers/piggyback.py M cmk/core_helpers/snmp.py M tests/unit/cmk/base/sources/test_ipmi_management_board_data_source.py M tests/unit/cmk/base/sources/test_piggy_back_data_source.py M tests/unit/cmk/base/sources/test_snmp_data_source.py M tests/unit/cmk/core_helpers/test_summarizers.py Log Message: ----------- Source success is independent of content of payload This is not only a code simplification, but also semantically correct: The success of the data source does not depend on the content of the payload (or the mode). It is not obvious to me what the "proper" solution for piggyback would be -- but it is not examining the payload. Change-Id: If0b3b5137b325750354e1aab62271bec756405ec