From 294eace024b0e6c34cb64e3bb4f8958bb98ee00b Mon Sep 17 00:00:00 2001 From: =?utf8?q?Zbigniew=20J=C4=99drzejewski-Szmek?= Date: Mon, 7 Jun 2021 13:29:19 +0200 Subject: [PATCH] core: disable "update" units in the initramfs Initially I wanted to add ConditionPathExists=!/etc/initrd-release in various units (ldconfig.service, systemd-sysusers.service, systemd-hwdb-update.service, systemd-journal-catalog-update, systemd-update-done.service), but I think it's better to just disable the mechanism in the initrd altogether. Initrd images are put together in a very particular way, and there is not need to do post-update steps on them. If a unit from some other package winds up in the initrd, we wouldn't want to invoke it either. Also, any modifications are ephemeral, so any update would happen on every use. And finally, initrd images are all about speed, and we shouldn't invoke any unneeded services. --- src/shared/condition.c | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/src/shared/condition.c b/src/shared/condition.c index d4d9eacc4bd..b86312548d0 100644 --- a/src/shared/condition.c +++ b/src/shared/condition.c @@ -675,6 +675,11 @@ static int condition_test_needs_update(Condition *c, char **env) { if (r > 0) return b; + if (in_initrd()) { + log_debug("We are in an initrd, not doing any updates."); + return false; + } + if (!path_is_absolute(c->parameter)) { log_debug("Specified condition parameter '%s' is not absolute, assuming an update is needed.", c->parameter); return true; -- 2.47.3