commit 10cd4393c98500ffc30200d8f503fad5968f0f4b
author Brad Bishop <bradleyb@fuzziesquirrel.com> Wed Nov 04 20:05:54 2020 -0500
committer Brad Bishop <bradleyb@fuzziesquirrel.com> Fri Nov 06 09:32:26 2020 -0500
tree 3aee32b30b5d7366e4618ce4a88a009adf6e58ec
parent c61ae35c3aa09960d816a8acc4f931274ac104db

entity-manager: systemd: use dbus instead of simple

Use systemd service type dbus instead of simple.  This avoids entity
manager client applications that have correctly ordered themselves after
entity manager from attempting to connect before it has claimed a bus
name.

When Type=dbus, systemd automatically adds an implicit dependency on
dbus.socket (so remove the unnecessary dependency on dbus.service).

Tested: systemctl restart xyz.openbmc_project.EntityManager works
Change-Id: I450b4ce28d87dfad9143b8b20cd9445e00dd07b0
Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>

service_files/xyz.openbmc_project.EntityManager.service

diff --git a/service_files/xyz.openbmc_project.EntityManager.service b/service_files/xyz.openbmc_project.EntityManager.service
index 651b0f2..448e61d 100644
--- a/service_files/xyz.openbmc_project.EntityManager.service
+++ b/service_files/xyz.openbmc_project.EntityManager.service
@@ -1,13 +1,13 @@
 [Unit]
 Description=Entity Manager
-After=dbus.service
 
 [Service]
 ExecStartPre=/bin/mkdir -p /var/configuration
 ExecStartPre=/bin/mkdir -p /tmp/overlays
 ExecStart=/usr/bin/entity-manager
 Restart=always
-Type=simple
+Type=dbus
+BusName=xyz.openbmc_project.EntityManager
 
 [Install]
 WantedBy=basic.target