From 23b1e8d087c9e8c5a2cdcc6a91510a4e7ca8f72f Mon Sep 17 00:00:00 2001 From: Lennart Poettering Date: Tue, 1 Feb 2022 18:11:04 +0100 Subject: [PATCH] units: we need systemd-journald.service from systemd-journal-flush.service This is a follow-up for d5ee050ffc9d413253932d9340ade8c8fb111092, and reintroduces a requirement dep from systemd-journal-flush.service onto systemd-journald.service, but a weaker one than originally: a Wants= one instead of a Requires= one. Why? Simply because the service issues an IPC call to the journald, hence it should pull it in. (Note that socket activation doesn't happen for the Varlink socket it uses, hence we should pull in the service itself.) --- units/systemd-journal-flush.service | 1 + 1 file changed, 1 insertion(+) diff --git a/units/systemd-journal-flush.service b/units/systemd-journal-flush.service index 6efb8734a7..5d0b811ae3 100644 --- a/units/systemd-journal-flush.service +++ b/units/systemd-journal-flush.service @@ -11,6 +11,7 @@ Description=Flush Journal to Persistent Storage Documentation=man:systemd-journald.service(8) man:journald.conf(5) DefaultDependencies=no +Wants=systemd-journald.service After=systemd-journald.service systemd-remount-fs.service Before=systemd-tmpfiles-setup.service RequiresMountsFor=/var/log/journal