[checkmk-commits] [tribe29/checkmk] 860e78: Pass path to PackedConfigStore

Branch: refs/heads/master Home: https://github.com/tribe29/checkmk Commit: 860e7834a38ba4a7e4bde13a5435925d39ca567e https://github.com/tribe29/checkmk/commit/860e7834a38ba4a7e4bde13a5435925d3… Author: Mathias Laurin <mathias.laurin(a)tribe29.com> Date: 2021-06-15 (Tue, 15 Jun 2021) 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: ----------- Pass path to PackedConfigStore This makes the signature of the store consistent with other stores and caches, such that the FileCache or the SectionStore. Indeed, what the store actually *wants* is a path where to read and write instead of a serial number that is changed immediately to a path. Moreover, the serial number is an implementation detail of the path that we should not be passing around too much. This patch adds a class method taking the serial in order to only incur minimal changes to the callers. Change-Id: Ia50a879b2a66797c8e3c2cc0f894ca4664fada43 Commit: 12c94275e92af4db168734a943cf7c5dad7c1923 https://github.com/tribe29/checkmk/commit/12c94275e92af4db168734a943cf7c5da… Author: Mathias Laurin <mathias.laurin(a)tribe29.com> Date: 2021-06-15 (Tue, 15 Jun 2021) Changed paths: M cmk/core_helpers/controller.py M cmk/core_helpers/crash_reporting.py Log Message: ----------- Fixup typing of one instance of ConfigSerial The argument to ConfigSerial() must be number-like: it is the serial. See Also: current_helper_config_serial(), next_helper_config_serial() Change-Id: I17fd8c38418d29137783c190b83efff865ec0223 Compare: https://github.com/tribe29/checkmk/compare/791ed5006687...12c94275e92a