[checkmk-commits] [tribe29/checkmk] d4e81c: Don't output logging information in automation calls

Branch: refs/heads/master Home: https://github.com/tribe29/checkmk Commit: d4e81cde0072341b1e23dba8490a2ecff09275ab https://github.com/tribe29/checkmk/commit/d4e81cde0072341b1e23dba8490a2ecff… Author: Tom Baerwinkel <tom.baerwinkel(a)tribe29.com> Date: 2020-09-24 (Thu, 24 Sep 2020) Changed paths: M cmk/base/events.py M cmk/base/modes/check_mk.py M cmk/base/notify.py M cmk/gui/wato/pages/notifications.py M cmk/utils/piggyback.py Log Message: ----------- Don't output logging information in automation calls Remove setup_console_logging in cmk.utils.piggyback. This leaves the initial NullHandler of cmk.utils.log.logger in place. This way no logging is set up accidentally for other modules on import of cmk.utils.piggyback or modules that import it like e.g. the prominent cmk.base.config. Finally, use clear_console_logging to disable logging in automation calls. In this case it lead to unwanted output in the notification-get-bulks automation call which lead to a crash of the notifacations page in the GUI. The logging is disabled after all arguments are successfully parsed and right before the automation handler is called. Remove an unnecessary argument to check_mk_local_automation along the way. CMK-5517 Change-Id: I2ea4ac0fcb57acf52226ffc2b5b3d4c7f8253445 Commit: 98dade4e9ef2ff91a98e7efb6b724ce5ad0df9e9 https://github.com/tribe29/checkmk/commit/98dade4e9ef2ff91a98e7efb6b724ce5a… Author: Tom Baerwinkel <tom.baerwinkel(a)tribe29.com> Date: 2020-09-24 (Thu, 24 Sep 2020) Changed paths: M cmk/base/notify.py Log Message: ----------- Plugins can be found for bulk notifications again CMK-5517 Change-Id: Ifd5596b7b8ee012bf2d0cb28f6884eb88995ac71 Commit: c2d9255e462d3b001457241c74308ad599a9c815 https://github.com/tribe29/checkmk/commit/c2d9255e462d3b001457241c74308ad59… Author: Tom Baerwinkel <tom.baerwinkel(a)tribe29.com> Date: 2020-09-24 (Thu, 24 Sep 2020) Changed paths: M cmk/base/notify.py Log Message: ----------- Improve typing in cmk.base.notify a bit CMK-5517 Change-Id: Ie69bb65f1ce574091152163296ce76331f92e85c Compare: https://github.com/tribe29/checkmk/compare/71d338b6ff51...c2d9255e462d