incus: change service type from notify to simple

Incusd does not call sd_notify(READY=1) — the Go source has no
systemd notification support in the main daemon (only in incus-agent,
the guest-side component). With Type=notify, systemd waits indefinitely
for the ready notification and the service stays in 'activating' state
even though incusd is fully operational.

This was not caught before the v7 uprev because the previous build
was cached via sstate or the service was tested manually.

Signed-off-by: Bruce Ashfield <bruce.ashfield@gmail.com>

1 files changed, 4 insertions, 3 deletions

diff --git a/recipes-containers/incus/files/incus.service b/recipes-containers/incus/files/incus.service
index 22f0eac6..12df9492 100644
--- a/recipes-containers/incus/files/incus.service
+++ b/recipes-containers/incus/files/incus.service
@@ -1,12 +1,13 @@
 [Unit]
 Description=Incus - Container and VM manager
 Documentation=https://linuxcontainers.org/incus/docs/main/
-After=network-online.target
+After=network-online.target incus.socket
 Wants=network-online.target
+Requires=incus.socket
 ConditionPathExists=/var/lib/incus
 
 [Service]
-Type=notify
+Type=simple
 ExecStart=/usr/sbin/incusd --group incus-admin
 ExecStop=/usr/sbin/incusd shutdown
 TimeoutStartSec=600
@@ -16,7 +17,7 @@ LimitNOFILE=1048576
 LimitNPROC=infinity
 TasksMax=infinity
 Delegate=yes
-KillMode=process
+KillMode=control-group
 
 [Install]
 WantedBy=multi-user.target