[checkmk-commits] [tribe29/checkmk] fe2f4d: piggyback summarizer: add repr

Branch: refs/heads/master Home: https://github.com/tribe29/checkmk Commit: fe2f4ddd961e63eff46926d21648b22c585d1a4b https://github.com/tribe29/checkmk/commit/fe2f4ddd961e63eff46926d21648b22c5… Author: Mathias Laurin <mathias.laurin(a)tribe29.com> Date: 2021-01-12 (Tue, 12 Jan 2021) Changed paths: M cmk/core_helpers/piggyback.py M tests/unit/cmk/core_helpers/test_summarizers.py Log Message: ----------- piggyback summarizer: add repr Add `__repr__` method and smoke tests to the PiggybackSummarizer. CMK-6842 Change-Id: If0782bea830aca3bd3cbebcbde67f1d1441bfb86 Commit: cd080617ef188e900199fce590cd1e0d7066773e https://github.com/tribe29/checkmk/commit/cd080617ef188e900199fce590cd1e0d7… Author: Mathias Laurin <mathias.laurin(a)tribe29.com> Date: 2021-01-12 (Tue, 12 Jan 2021) Changed paths: M cmk/core_helpers/piggyback.py M cmk/utils/piggyback.py Log Message: ----------- Fix typing for get_piggyback_raw_data The function already handles a falsy `piggybacked_hostname`. We might as well make the argument optional. CMK-6842 Change-Id: Ib5a6f4a2ab4e7fd58bcaa769ef70e60f34d67c45 Commit: 30a466ebfcd461bed4b1fefa5558f674f06dfb6f https://github.com/tribe29/checkmk/commit/30a466ebfcd461bed4b1fefa5558f674f… Author: Mathias Laurin <mathias.laurin(a)tribe29.com> Date: 2021-01-12 (Tue, 12 Jan 2021) Changed paths: M cmk/core_helpers/piggyback.py M tests/unit/cmk/core_helpers/test_summarizers.py Log Message: ----------- Properly handle missing data in piggyback summarizer The 'missing data' error was triggered when the piggyback file was missing either from the directory with the hostname or from the directory with the IP address. The error should only be triggered when *both* are missing. This patch fixes the logic and adds tests for this very case. CMK-6842 Change-Id: I92f5d1fe98777002dec80e81abfee225eb23c6ef Compare: https://github.com/tribe29/checkmk/compare/f458987cebe7...30a466ebfcd4