Merge pull request #22 from hemanthnakkina/fix-layer-definition

Add requires to Notifier service
openstack-charmers · Nov 27, 2024 · 0f43eb2 · 0f43eb2
2 parents ce7737e + 000be3e
commit 0f43eb2
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",