[checkmk-commits] [tribe29/checkmk] 0ff2e0: refactor: inline a couple of functions

Branch: refs/heads/master Home: https://github.com/tribe29/checkmk Commit: 0ff2e0b56143ef0d6f1e68e5e19e6076e765d85e https://github.com/tribe29/checkmk/commit/0ff2e0b56143ef0d6f1e68e5e19e6076e… Author: Moritz Kiemer &lt;moritz.kiemer(a)tribe29.com&gt; Date: 2021-11-29 (Mon, 29 Nov 2021) Changed paths: M cmk/core_helpers/tcp.py M tests/unit/cmk/core_helpers/test_fetchers.py Log Message: ----------- refactor: inline a couple of functions Change-Id: I88514f0ef862ea56a948e48c62c95432cf99efee Commit: b4e0ffc49085cfb3f6955daa5ee727eefd59a4f6 https://github.com/tribe29/checkmk/commit/b4e0ffc49085cfb3f6955daa5ee727eef… Author: Moritz Kiemer &lt;moritz.kiemer(a)tribe29.com&gt; Date: 2021-11-29 (Mon, 29 Nov 2021) Changed paths: M cmk/core_helpers/tcp.py Log Message: ----------- simplify failed decryption case 'decrypt_by_agent_protocol' will not throw an MKFetcherError. If we did not detect the transport protocol to be 'PLAIN' but neither succeeded in decrypting the output, we raise unconditionally. We no longer assume that we might have misinterpreted plaintext data as being encrypted (because of missing '<<<' at the beginning). Change-Id: I24154bde82d92db8b5bc544d24b650ed13dfdf90 Commit: 3ca33d16607e92c4de73b9701a43fd31a03589fc https://github.com/tribe29/checkmk/commit/3ca33d16607e92c4de73b9701a43fd31a… Author: Moritz Kiemer &lt;moritz.kiemer(a)tribe29.com&gt; Date: 2021-11-29 (Mon, 29 Nov 2021) Changed paths: M cmk/core_helpers/tcp.py M tests/unit/cmk/core_helpers/test_fetchers.py Log Message: ----------- tcp fetcher: centralize transport validation This commit is mostly a refactoring, bundling up the logging and the validation of the detected transport protocol, respectively. If we're using TLS, we accept both encrypted or unencrypted data. This is done to ease migration from 2.0 to 2.1. If deployed via agent bakery the agent will not additionaly encrypt the data below the TLS layer. Change-Id: I33ccb28c41cc03c2d4c612ba6974a92764cba8d8 Commit: c86d3c5f3d77d5b67f57ca4c266a111ad46ce451 https://github.com/tribe29/checkmk/commit/c86d3c5f3d77d5b67f57ca4c266a111ad… Author: Moritz Kiemer &lt;moritz.kiemer(a)tribe29.com&gt; Date: 2021-11-29 (Mon, 29 Nov 2021) Changed paths: M cmk/core_helpers/tcp.py Log Message: ----------- replace cryptic SSL error message Change-Id: I2138608660ee90116f2fcd9d6780374f9c2e47e5 Commit: cb8081d4c2ef2ae02c5f1c38c040a16d34fa98e3 https://github.com/tribe29/checkmk/commit/cb8081d4c2ef2ae02c5f1c38c040a16d3… Author: Moritz Kiemer &lt;moritz.kiemer(a)tribe29.com&gt; Date: 2021-11-29 (Mon, 29 Nov 2021) Changed paths: M cmk/gui/plugins/wato/check_mk_configuration.py Log Message: ----------- allow users to enfoce TLS Change-Id: I2452fcb4773fd40b99a439ae6977dc8fb0c3b96e Commit: 2e18cb8d7efe2d1f3ee1f68df720280f2e59a709 https://github.com/tribe29/checkmk/commit/2e18cb8d7efe2d1f3ee1f68df720280f2… Author: Moritz Kiemer &lt;moritz.kiemer(a)tribe29.com&gt; Date: 2021-11-29 (Mon, 29 Nov 2021) Changed paths: M cmk/base/agent_based/checking/__init__.py M cmk/base/agent_based/decorator.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/utils/check_utils.py M tests/unit/cmk/base/agent_based/discovery/test_discovery.py M tests/unit/cmk/base/agent_based/test_utils.py M tests/unit/cmk/utils/test_utils_check_utils.py Log Message: ----------- Make ActiveCheckResult a dataclass This is allows to add default values. Also we try to avoid the "str" vs. "Sequence[str]" ambiguity. Change-Id: If0fd4eb86bf23f9c41fbe7b94071f62dbd9bb7e6 Commit: 11916d3451911e4599c6ebca5d2a0b56222939c2 https://github.com/tribe29/checkmk/commit/11916d3451911e4599c6ebca5d2a0b562… Author: Moritz Kiemer &lt;moritz.kiemer(a)tribe29.com&gt; Date: 2021-11-29 (Mon, 29 Nov 2021) Changed paths: M cmk/core_helpers/agent.py M tests/unit/cmk/base/sources/test_tcp_data_source.py M tests/unit/cmk/core_helpers/test_summarizers.py Log Message: ----------- summarizer: use ActiveCheckResult This will allow us to aggregate check results containing details (everyhing after the first newline) more easily. It also fixes the wrong aggregation (we used max, but CRIT should be considered worse than UNKNOWN). Change-Id: If2aecb68fcaedd17af5e9bef4baa820b49543094 Commit: 8f0d9de1541313ef6d384228753d0b0698207673 https://github.com/tribe29/checkmk/commit/8f0d9de1541313ef6d384228753d0b069… Author: Frans Fürst &lt;frans.fuerst(a)tribe29.com&gt; Date: 2021-11-29 (Mon, 29 Nov 2021) Changed paths: M buildscripts/scripts/build-build-containers.jenkins Log Message: ----------- let build containers bootstrap from IMAGE_UBUNTU_20_04 rather than IMAGE_TESTING in order to avoid a circular dependency Change-Id: Id0910a158910e7c812e04f2743ca5ab9760b0aa5 Compare: https://github.com/tribe29/checkmk/compare/d5d3608c4e8f...8f0d9de15413