[checkmk-commits] [Checkmk/checkmk] 5eb725: piggyback: consolidate abandonded cases

Branch: refs/heads/master Home: https://github.com/Checkmk/checkmk Commit: 5eb7257e1ae7f599c2a87525e861ecd574c6dc66 https://github.com/Checkmk/checkmk/commit/5eb7257e1ae7f599c2a87525e861ecd57… Author: Moritz Kiemer <moritz.kiemer(a)checkmk.com> Date: 2024-05-21 (Tue, 21 May 2024) Changed paths: M cmk/utils/piggyback.py M tests/unit/cmk/utils/test_piggyback.py Log Message: ----------- piggyback: consolidate abandonded cases The difference between these to cases is not very clear from the messages anyway. But even if it was: If we want to monitor whether a given source host is sending piggyback data *at all*, we should do that on the source host, (e.g. by a dedicated service) not on the target host(s). Change-Id: I86d2ef053b0cc54ec6df847d61502541749d1733 Commit: eff8d38b1a6a8bb92ea2733acf0a5da84132aca6 https://github.com/Checkmk/checkmk/commit/eff8d38b1a6a8bb92ea2733acf0a5da84… Author: Moritz Kiemer <moritz.kiemer(a)checkmk.com> Date: 2024-05-21 (Tue, 21 May 2024) Changed paths: M cmk/fetchers/_piggyback.py M cmk/utils/piggyback.py M tests/unit/cmk/utils/test_piggyback.py Log Message: ----------- piggyback: kw only + rename Change-Id: Ic61c2f9d535c1a6b9c1bc9cff2ea26a85c539c25 Commit: 17c2c25be8168a814d0409a97e22825af99c7728 https://github.com/Checkmk/checkmk/commit/17c2c25be8168a814d0409a97e22825af… Author: Moritz Kiemer <moritz.kiemer(a)checkmk.com> Date: 2024-05-21 (Tue, 21 May 2024) Changed paths: M cmk/utils/piggyback.py Log Message: ----------- piggyback: compute some meta infos earlier This avoids one required race condition handling when computing the files age. Change-Id: Ib6ad673100e0f221d7c1cdd0fd032d0234a6da8b Commit: 4a80c1aef3b39d4935544e18276f67c8ac116911 https://github.com/Checkmk/checkmk/commit/4a80c1aef3b39d4935544e18276f67c8a… Author: Moritz Kiemer <moritz.kiemer(a)checkmk.com> Date: 2024-05-21 (Tue, 21 May 2024) Changed paths: M cmk/utils/piggyback.py Log Message: ----------- piggyback: simplify error handling The case of a missing file is a simple race condition, that can be ignored. All other cases are sufficiently unexpected to cause the fetcher to fail. Note that the race condition would only have been reported by the _summarizer_, meaning that the fetcher actually did _not_ observe the same. Change-Id: I37b79e4a4fe0cc157c48c4f9b33124fb2b821892 Compare: https://github.com/Checkmk/checkmk/compare/5e1718a52994...4a80c1aef3b3 To unsubscribe from these emails, change your notification settings at https://github.com/Checkmk/checkmk/settings/notifications