[checkmk-commits] [tribe29/checkmk] c5f375: Remove dead code in store.py

Branch: refs/heads/1.6.0 Home: https://github.com/tribe29/checkmk Commit: c5f375cc8883a4a877e580fb50f0bd27d4a417ad https://github.com/tribe29/checkmk/commit/c5f375cc8883a4a877e580fb50f0bd27d… Author: Tom Baerwinkel <tom.baerwinkel(a)tribe29.com> Date: 2019-07-09 (Tue, 09 Jul 2019) Changed paths: M cmk/utils/store.py Log Message: ----------- Remove dead code in store.py Change-Id: Ib06f9efb8417a884305eaeaceee0db50d2a96f8b Commit: 6f9d2a3cb34f292152e92535292901ebb653a00f https://github.com/tribe29/checkmk/commit/6f9d2a3cb34f292152e92535292901ebb… Author: Tom Baerwinkel <tom.baerwinkel(a)tribe29.com> Date: 2019-07-09 (Tue, 09 Jul 2019) Changed paths: M cmk/gui/wato/__init__.py M cmk/gui/wato/pages/automation.py M cmk/gui/wato/pages/parentscan.py M cmk/gui/watolib/__init__.py M cmk/gui/watolib/activate_changes.py M cmk/gui/watolib/bulk_discovery.py M cmk/gui/watolib/utils.py M cmk/gui/webapi.py Log Message: ----------- Rename exclusive_lock to lock_checkmk_configuration CMK-2089 Change-Id: Icd44cea10bf207c48da960daf128b44578e3b1de Commit: f7863e3dd37fef56be18c5836ee828ed3411e8d1 https://github.com/tribe29/checkmk/commit/f7863e3dd37fef56be18c5836ee828ed3… Author: Tom Baerwinkel <tom.baerwinkel(a)tribe29.com> Date: 2019-07-09 (Tue, 09 Jul 2019) Changed paths: M cmk/gui/plugins/wato/utils/__init__.py M cmk/gui/wato/__init__.py M cmk/gui/wato/pages/automation.py M cmk/gui/wato/pages/parentscan.py M cmk/gui/watolib/__init__.py M cmk/gui/watolib/activate_changes.py M cmk/gui/watolib/bulk_discovery.py M cmk/gui/watolib/utils.py M cmk/gui/webapi.py M cmk/utils/store.py Log Message: ----------- Move Checkmk configuration locks to cmk.utils.store The functions lock_checkmk_configuration and lock_exclusive are now available in cmk.utils.store so that they can be used by cmk_base as well. This is e.g. useful for automation calls. CMK-2089 Change-Id: I44fb86e4b3f8174732b61eea322fc9c656dbbcb3 Commit: 8465c5c008a85db47baa5d6a0eaa614296ada4d6 https://github.com/tribe29/checkmk/commit/8465c5c008a85db47baa5d6a0eaa61429… Author: Tom Baerwinkel <tom.baerwinkel(a)tribe29.com> Date: 2019-07-09 (Tue, 09 Jul 2019) Changed paths: M cmk/gui/watolib/sites.py M cmk/utils/store.py M cmk_base/modes/check_mk.py Log Message: ----------- Protect snapshot unpacking, cmk --notify and the mknotifyd with a lock When a snapshot on a remote site is unpacked lock_checkmk_configuration is now used to avoid further modifications of the configuration. For the same reason the configuration reload of the mknotifyd and the command line cmk --notify are now protected with the configuration lock. The mknotifyd only tries to aquire the lock to remain operational if the lock is not available immediately. In this case the mknotifyd will use the old configuration for another cycle. CMK-2089 Change-Id: Ib52c0bcc60fbe50a51cf3994075fae9ef6f8f47b Commit: d5ace391f9513af5560e10cdc60827e7463a256e https://github.com/tribe29/checkmk/commit/d5ace391f9513af5560e10cdc60827e74… Author: Tom Baerwinkel <tom.baerwinkel(a)tribe29.com> Date: 2019-07-09 (Tue, 09 Jul 2019) Changed paths: A .werks/8780 Log Message: ----------- 8780 FIX mknotifyd: fix race condition in distributed setups In a distributed setup a slave site may try to deliver notifications locally during an activate changes even though the slave site should forward notifications to the master site. This effect appears when the option "Notification Spooling" is set to "Asynchronous local delivery by notification spooler" in the global settings and to "Forward to remote site by notification spooler" in the "Site specific global settings" of the slave site. This werk fixes this issue by locking the Checkmk configuration when a configuration snapshot is installed on the remote site. This way the "cmk --notify" command and the mknotifyd cannot use a partially installed Checkmk configuration. As a workaround the "Notification Spooling" can be set to "Forward to remote site by notification spooler" in the global settings and to "Asynchronous local delivery by Notification spooler" in the "Site specific global settings" of the slave site. CMK-2089 Change-Id: I40d88c47ec078b78963787a3de7dbc53d6955b28 Compare: https://github.com/tribe29/checkmk/compare/f9ae468e9430...d5ace391f951