[checkmk-commits] [tribe29/checkmk] 2a2a60: Cleanup livestatus usage from RBN processing

Branch: refs/heads/2.1.0 Home: https://github.com/tribe29/checkmk Commit: 2a2a6000ecd35a2bf97e464616df106da84aa693 https://github.com/tribe29/checkmk/commit/2a2a6000ecd35a2bf97e464616df106da… Author: Lars Michelsen &lt;lm(a)tribe29.com&gt; Date: 2022-07-19 (Tue, 19 Jul 2022) Changed paths: M cmk/base/notify.py M tests/unit/cmk/base/test_notify.py Log Message: ----------- Cleanup livestatus usage from RBN processing In case a user configured the contact selection option "Members of contact groups" (contact_groups), a Livestatus query was made to get the members of the configured contact groups. This was done every time such a rule matched a notification. In specific situations such a lookup could cause dead locks between the Microcore and the notify helper. Since the notification helpers have the whole base configuration in memory, we can simply lookup the group memberships without the Livestatus lookup. Change-Id: Ia78b26153ea53ea39e3e7e4be5922e93eab268d8 Commit: 1842cef032abb988b6b32c424efe99ff28fcab72 https://github.com/tribe29/checkmk/commit/1842cef032abb988b6b32c424efe99ff2… Author: Lars Michelsen &lt;lm(a)tribe29.com&gt; Date: 2022-07-19 (Tue, 19 Jul 2022) Changed paths: A .werks/14285 Log Message: ----------- 14285 FIX Fix frozen Microcore (Livestatus not responding) during config reloads The reload of the Microcore core and it's helper processed could freeze when the core had notifications pending during reload. This was caused by a deadlock between the Notification helper and the Microcore. The Microcore was still alive but waiting to the notification helper to terminate while the notification helper waited for the Microcore. while the cmc.log showed a message like: <tt>still X unsent events, sending them now</tt>. Change-Id: I8ac1a2a179e44f761c21a7a7a3e0ebb3ab4cc976 Commit: a4b509ece12f374b79239a4276af4ec30e1be6a7 https://github.com/tribe29/checkmk/commit/a4b509ece12f374b79239a4276af4ec30… Author: Lars Michelsen &lt;lm(a)tribe29.com&gt; Date: 2022-07-19 (Tue, 19 Jul 2022) Changed paths: M cmk/base/notify.py Log Message: ----------- Apply connect timeout to local livestatus connections In case the core is not responsive for some reason, then it is better to continue without sending the command and logging it instead of blocking endlessly. Change-Id: Ie6f6667053b6bd2423dee40f829656725aeb0bdb Commit: 8ccae23d9a02e629c86004ab0a3b9762ed512c24 https://github.com/tribe29/checkmk/commit/8ccae23d9a02e629c86004ab0a3b9762e… Author: Lars Michelsen &lt;lm(a)tribe29.com&gt; Date: 2022-07-19 (Tue, 19 Jul 2022) Changed paths: A .werks/14286 M cmk/notification_plugins/mail.py Log Message: ----------- 14286 FIX mail: Add timeout to graph processing of HTML mail notifications In case fetching the graphs takes more than 10 seconds, the notification plugin aborts waiting for the graphs and continues to send the notification without graphs. Change-Id: I207d83f440fc2b0c34fbb8175db254baf2ef3226 Commit: 9a68d80fd1746839851166236ac1f4990568e329 https://github.com/tribe29/checkmk/commit/9a68d80fd1746839851166236ac1f4990… Author: Lars Michelsen &lt;lm(a)tribe29.com&gt; Date: 2022-07-19 (Tue, 19 Jul 2022) Changed paths: M tests/integration/cmk/base/test_automations.py Log Message: ----------- Fix integration test after 1e0b49acabef83f2ce66be34ef773ec8c2e85b31 Change-Id: Ie217e8a3c964c7d2b1982402fb7bc781b97ee504 Compare: https://github.com/tribe29/checkmk/compare/6ea5cc167033...9a68d80fd174