[checkmk-commits] [tribe29/checkmk] 01bbc3: Extract persistence of packed config

Branch: refs/heads/master Home: https://github.com/tribe29/checkmk Commit: 01bbc302fae8ff043b6faf4faf5298eb7275b3b2 https://github.com/tribe29/checkmk/commit/01bbc302fae8ff043b6faf4faf5298eb7… Author: Lars Michelsen <lm(a)tribe29.com> Date: 2020-09-07 (Mon, 07 Sep 2020) Changed paths: M cmk/base/config.py M tests/unit/cmk/base/test_config.py Log Message: ----------- Extract persistence of packed config Untie the packed config calculation from the persistence logic and add unit tests for the extracted persistence logic. Change-Id: Iec06939a65d0c38184c0a820547dad9182e58035 Commit: 22f317ed359f79249235ebe3b21e4395a6be76ea https://github.com/tribe29/checkmk/commit/22f317ed359f79249235ebe3b21e4395a… Author: Lars Michelsen <lm(a)tribe29.com> Date: 2020-09-07 (Mon, 07 Sep 2020) Changed paths: M cmk/base/config.py Log Message: ----------- PackedConfigStore: Refactor to use pathlib paths Change-Id: Ica98844e25d499450ba8f1fe824589fc489d8f03 Commit: a46294de9ee4d2172dd9a3cdb6b19378ca525ac6 https://github.com/tribe29/checkmk/commit/a46294de9ee4d2172dd9a3cdb6b19378c… Author: Lars Michelsen <lm(a)tribe29.com> Date: 2020-09-07 (Mon, 07 Sep 2020) Changed paths: M cmk/base/config.py M tests/unit/cmk/base/test_config.py Log Message: ----------- Extract packed config loading from PackedConfig to existing load function Loading the packed config does not need the PackedConfig class. We just need to use the PackedConfigStore to read the current packed config and merge the read definitions into the cmk.base.config. It is not necessary to involve the PackedConfig class. The left PackedConfig class is now a generator of the packed config content. The next commit will clean up naming and the structure to clarify this. Change-Id: I3abe41fa9d13b2e4b1f46640a8a48995d461c3ef Commit: ce2468b1ce104c50fd227d664d929e3b48922606 https://github.com/tribe29/checkmk/commit/ce2468b1ce104c50fd227d664d929e3b4… Author: Lars Michelsen <lm(a)tribe29.com> Date: 2020-09-07 (Mon, 07 Sep 2020) Changed paths: M cmk/base/config.py M cmk/base/core_nagios.py M tests/unit/cmk/base/test_config.py Log Message: ----------- Centralize packed config saving call in config.save_packed_config * Make config generator more testable (handing over ConfigCache) * Add simple test for saving of packed config Change-Id: Ie987e4674b99e1a573203004e44d54874d905445 Commit: 851d863a772b10ddc0d0de2bb3a3877a9b01950e https://github.com/tribe29/checkmk/commit/851d863a772b10ddc0d0de2bb3a3877a9… Author: Lars Michelsen <lm(a)tribe29.com> Date: 2020-09-07 (Mon, 07 Sep 2020) Changed paths: M cmk/base/config.py Log Message: ----------- Clear separation of packed config generation and writing Change-Id: Iee907b98cfe156334160f93bbf2d815f4c21b545 Commit: 8e7a1da0670df35db720772319a64016cb4a3aba https://github.com/tribe29/checkmk/commit/8e7a1da0670df35db720772319a64016c… Author: Lars Michelsen <lm(a)tribe29.com> Date: 2020-09-07 (Mon, 07 Sep 2020) Changed paths: M cmk/base/config.py M cmk/base/core_config.py M cmk/base/core_nagios.py M cmk/base/modes/check_mk.py M tests/unit/cmk/base/test_config.py Log Message: ----------- Use optional config serial in packed config persistence * All call sites that are aware of the config serial can now explicitly load a specific config by using it's config serial. * All other call sites can use "None" as config serial to load the latest of the available configs. This is required for compatibility with the current code and needed for most of the call sites. All of these call sites may be updated in the future to be aware of the config serials, but this is beyond the scope of our current development. Please note: The "latest" link is not written at the moment. This will be added in the next commit. Change-Id: I250a23a9aa68e3d71ca92a3a5eca3bbe4cdab827 Commit: fb499debcf573cfc5e79c1c96c894e01cc4050a2 https://github.com/tribe29/checkmk/commit/fb499debcf573cfc5e79c1c96c894e01c… Author: Lars Michelsen <lm(a)tribe29.com> Date: 2020-09-07 (Mon, 07 Sep 2020) Changed paths: M cmk/base/config.py M tests/unit/cmk/base/test_config.py Log Message: ----------- Write packed config link to latest serial There are some "load packed config" calls that are not aware of the current config serial. In the past these call sites always loaded the latest version of the packed config. We preserve this behaviour by generating a symlink "latest" that always points to the last packed config version. This makes the unversioned load config calls work as before until we migrate them to be versioned. Change-Id: Ia89b06a00921d2a04bfb19958d636f17b5d0bae5 Compare: https://github.com/tribe29/checkmk/compare/5a02df1ef183...fb499debcf57