[checkmk-commits] [tribe29/checkmk] e72557: PackedConfig: Separate config generation and seria...

Branch: refs/heads/2.0.0 Home: https://github.com/tribe29/checkmk Commit: e725576317b36a6e30cd7582da09bd8a46a3964c https://github.com/tribe29/checkmk/commit/e725576317b36a6e30cd7582da09bd8a4… Author: Mathias Laurin <mathias.laurin(a)tribe29.com> Date: 2020-12-24 (Thu, 24 Dec 2020) Changed paths: M cmk/base/config.py M tests/unit/cmk/base/test_config.py Log Message: ----------- PackedConfig: Separate config generation and serialization This makes the requirements on the `PackedConfigStore` explicit. We also now fully encapsulate the serialization scheme of the packed config. This should let us use a cheaper one because `repr()`, `eval()`, and `compile()` is too expensive. Change-Id: I1e4e957cdf10a781689459aa90f0113dbd4dc2a3 Commit: 3b0bad715e3f11d63556751f1cac62ec6743196e https://github.com/tribe29/checkmk/commit/3b0bad715e3f11d63556751f1cac62ec6… Author: Mathias Laurin <mathias.laurin(a)tribe29.com> Date: 2020-12-24 (Thu, 24 Dec 2020) Changed paths: M cmk/base/config.py M tests/unit/cmk/base/test_config.py Log Message: ----------- Let PackedConfigStore.write take a mapping `PackedConfigStore.read()` returns a mapping so that taking a mapping on `write()` makes the class internally consistent. This also makes it clear that there cannot be two keys with the same name. CMK-6846 Change-Id: I417d8525fd62ae1f1dade77d7b66ed849c6ad4aa Commit: dc83aac374053d170ff65c5159011d8d0a167016 https://github.com/tribe29/checkmk/commit/dc83aac374053d170ff65c5159011d8d0… Author: Mathias Laurin <mathias.laurin(a)tribe29.com> Date: 2020-12-24 (Thu, 24 Dec 2020) Changed paths: M cmk/base/config.py M tests/unit/cmk/base/test_config.py M tests/unit/cmk/base/test_core_config.py Log Message: ----------- Serialize the packed config with pickle The data is now serialized only once using pickle and the clear text version is not saved anymore. Replacing the double serialization (to string as well as repr/eval/marshal) represents a factor 35+ speed improvement on a small test set. We expect this factor to scale well to larger test sets as well. CMK-6846 Change-Id: Icbeceadb3da63e58f10130dd4cb03c844d90a7a9 Compare: https://github.com/tribe29/checkmk/compare/560a2bce1b5e...dc83aac37405