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

Branch: refs/heads/master Home: https://github.com/tribe29/checkmk Commit: 99efeafcdf776a006aaa23267af8e2451a970509 https://github.com/tribe29/checkmk/commit/99efeafcdf776a006aaa23267af8e2451… 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: f813d100e86c32beef8db76f3641f280037423de https://github.com/tribe29/checkmk/commit/f813d100e86c32beef8db76f3641f2800… 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: 68f114487991a1c152e3b7ac17f704159b3d65db https://github.com/tribe29/checkmk/commit/68f114487991a1c152e3b7ac17f704159… 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: 4bc0b1e03e3c684a28fa6469502382d25fe8e2cc https://github.com/tribe29/checkmk/commit/4bc0b1e03e3c684a28fa6469502382d25… 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: 250db039d94324808e3145a31a1f23f1c6cce98d https://github.com/tribe29/checkmk/commit/250db039d94324808e3145a31a1f23f1c… 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/c39d370e6ba6...250db039d943