Add requires to Notifier service

Autostart of mix of service dependencies resulted in panic mode pebble, see bug [1] As a workaround, add Requires to notifier service definition. [1] canonical/pebble#525
openstack-charmers · Nov 27, 2024 · 000be3e · 000be3e
1 parent ce7737e
commit 000be3e
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 0 deletions.
diff --git a/src/charm.py b/src/charm.py
@@ -306,6 +306,9 @@ def _rabbitmq_layer(self) -> dict:
                     "summary": "Pebble notifier",
                     "command": "/usr/bin/notifier",
                     "startup": "enabled",
+                    # Workaround to avoid bug
+                    # https://github.com/canonical/pebble/issues/525
+                    "requires": [RABBITMQ_SERVICE],
                 },
             },
         }

diff --git a/tests/unit/test_charm.py b/tests/unit/test_charm.py
@@ -90,6 +90,7 @@ def test_rabbitmq_pebble_ready(self):
                     "override": "replace",
                     "startup": "enabled",
                     "summary": "Pebble notifier",
+                    "requires": ["rabbitmq"],
                 },
                 "epmd": {
                     "override": "replace",