[checkmk-commits] [tribe29/checkmk] 406eca: Revert reecent AppendStore changes for now.

Branch: refs/heads/master Home: https://github.com/tribe29/checkmk Commit: 406ecaa0ce306b5ea6490f18fe46fad1f1b5a148 https://github.com/tribe29/checkmk/commit/406ecaa0ce306b5ea6490f18fe46fad1f… Author: Sven Panne <sven.panne(a)tribe29.com> Date: 2023-03-08 (Wed, 08 Mar 2023) Changed paths: M cmk/gui/plugins/wato/remote_audit_logs.py M cmk/gui/watolib/activate_changes.py M cmk/gui/watolib/appendstore.py M tests/unit/cmk/gui/watolib/test_changes.py Log Message: ----------- Revert reecent AppendStore changes for now. Revert "Mechanical refactoring: Move locking to caller." This reverts commit 77d7d070586eca68c9589b3e8850bea5c8ff6544. (This just sits on top of the probably problematic change.) Revert "Remove write() from AppendStore's API." This reverts commit 0f4eec0e0bf4508b1b94875b031c63e35759cdd9. (This seems to be the problematic change.) Commit: 8592a965d3bf4fd72d5c474959c2e8bd35a39f13 https://github.com/tribe29/checkmk/commit/8592a965d3bf4fd72d5c474959c2e8bd3… Author: Lisa Pichler <lisa.pichler(a)tribe29.com> Date: 2023-03-08 (Wed, 08 Mar 2023) Changed paths: M cmk/gui/plugins/metrics/fs.py Log Message: ----------- filesystem metrics: fix fs_free not displayed in graph CMK-12488 Change-Id: Ic3f220639e0f5ea530ae266e1fce9f727f2e6c66 Commit: 1be2e825de5e4543e10f958ff4c9bf38b8a5e5b5 https://github.com/tribe29/checkmk/commit/1be2e825de5e4543e10f958ff4c9bf38b… Author: Mathias Laurin <mathias.laurin(a)tribe29.com> Date: 2023-03-08 (Wed, 08 Mar 2023) Changed paths: M cmk/base/agent_based/data_provider.py Log Message: ----------- Inline trivial function Change-Id: I4974a0668c1913a12eec180588d0f152fe1a1867 Commit: 9ce5fbf01d9175de9c6e4f52c6720465bf4696c4 https://github.com/tribe29/checkmk/commit/9ce5fbf01d9175de9c6e4f52c6720465b… Author: Mathias Laurin <mathias.laurin(a)tribe29.com> Date: 2023-03-08 (Wed, 08 Mar 2023) Changed paths: M cmk/base/agent_based/data_provider.py M tests/unit/cmk/base/agent_based/test_data_provider_parsed_sections_resolver.py M tests/unit/cmk/base/agent_based/test_data_provider_sections_parser.py Log Message: ----------- Remove SectionParser tuple Firing a new abstraction for a two-tuple isn't really worth it. Moreover, handling the name separately from the business logic is advantageous in many cases as they tend to follow different code paths---as can be seen here as well where most of the logic is performed on the name only. See also: https://wiki.haskell.org/Fairbairn_threshold Change-Id: I63b9ab4a5a4e84c2f3a41427ec11405d05a3dd96 Commit: d6696ca2bb4143674ccd51c0d77e86d522bce6dd https://github.com/tribe29/checkmk/commit/d6696ca2bb4143674ccd51c0d77e86d52… Author: Lukas Lengler <lukas.lengler(a)tribe29.com> Date: 2023-03-08 (Wed, 08 Mar 2023) Changed paths: M cmk/base/plugins/agent_based/esx_vsphere_vm.py Log Message: ----------- Fix creation of "cmk/vsphere_object:vm" HostLabel for vCenter VMs CMK-12541 Change-Id: If0a3816490146936be43091965cadfa3f4f39eee Commit: 98b48f2d6d04bd161f0cdcdb16c283148bfd1935 https://github.com/tribe29/checkmk/commit/98b48f2d6d04bd161f0cdcdb16c283148… Author: Sven Panne <sven.panne(a)tribe29.com> Date: 2023-03-08 (Wed, 08 Mar 2023) Changed paths: M cmk/gui/watolib/appendstore.py M tests/unit/cmk/gui/watolib/test_changes.py Log Message: ----------- Added mutable_view() to ABCAppendStore. This is intended to replace transform(), which is a bit inconvenient to use with our current code base. Change-Id: I9466b5b5b542ca2f7bd7dbe374ff3d9f0d6a7ddb Commit: 1865bdef475fb6948ee6fc547202acfaa362c92d https://github.com/tribe29/checkmk/commit/1865bdef475fb6948ee6fc547202acfaa… Author: Sven Panne <sven.panne(a)tribe29.com> Date: 2023-03-08 (Wed, 08 Mar 2023) Changed paths: M cmk/gui/watolib/activate_changes.py Log Message: ----------- Use mutable_view() in _confirm_activated_changes(). Change-Id: I8cac6c6f922ba7ff03e7698c2360bc0cd25b5da3 Compare: https://github.com/tribe29/checkmk/compare/733177a1234f...1865bdef475f