From: "Alex Kiernan" <alex.kiernan@gmail.com>
To: openembedded-core@lists.openembedded.org
Cc: Alex Kiernan <alex.kiernan@gmail.com>,
Ross Burton <ross.burton@intel.com>
Subject: [OE-Core][PATCH 4/6] run-postinsts: Set RemainAfterExit on systemd unit
Date: Wed, 22 Apr 2020 14:52:09 +0100 [thread overview]
Message-ID: <20200422135211.121292-4-alex.kiernan@gmail.com> (raw)
In-Reply-To: <20200422135211.121292-1-alex.kiernan@gmail.com>
run-postinsts is only expected to run once, but during startup, but if
any dependency is pulled into a transaction, even once it has been
marked disabled, then it can be restarted.
This leads to occasional failures during QA if an ssh session starts
whilst the existing transaction is still running:
Finished Run pending postinsts.
run-postinsts.service: Succeeded.
Condition check resulted in Commit a transient machine-id on disk being skipped.
Condition check resulted in Bind mount volatile /srv being skipped.
Condition check resulted in Bind mount volatile /var/spool being skipped.
Condition check resulted in Bind mount volatile /var/lib being skipped.
Condition check resulted in Bind mount volatile /var/cache being skipped.
Condition check resulted in Platform Persistent Storage Archival being skipped.
Condition check resulted in Rebuild Hardware Database being skipped.
Starting Run pending postinsts...
Condition check resulted in Kernel Configuration File System being skipped.
Condition check resulted in FUSE Control File System being skipped.
Condition check resulted in Load Kernel Modules being skipped.
Condition check resulted in File System Check on Root Device being skipped.
Condition check resulted in Huge Pages File System being skipped.
Condition check resulted in Journal Audit Socket being skipped.
dropbear@125-192.168.7.2:22-192.168.7.1:44226.service: Succeeded.
Condition check resulted in Platform Persistent Storage Archival being skipped.
Started SSH Per-Connection Server (192.168.7.1:44226).
dropbear@124-192.168.7.2:22-192.168.7.1:44224.service: Succeeded.
Started SSH Per-Connection Server (192.168.7.1:44224).
Condition check resulted in Commit a transient machine-id on disk being skipped.
Condition check resulted in Bind mount volatile /srv being skipped.
Condition check resulted in Bind mount volatile /var/spool being skipped.
Condition check resulted in Bind mount volatile /var/lib being skipped.
Condition check resulted in Bind mount volatile /var/cache being skipped.
Condition check resulted in Platform Persistent Storage Archival being skipped.
Condition check resulted in Rebuild Hardware Database being skipped.
Failed to start Run pending postinsts.
run-postinsts.service: Failed with result 'start-limit-hit'.
run-postinsts.service: Start request repeated too quickly.
Setting RemainAfterExit ensures that the unit remains active and is not
gratuitously restarted, unless done so explicitly using systemctl
restart.
Signed-off-by: Alex Kiernan <alex.kiernan@gmail.com>
---
.../run-postinsts/run-postinsts/run-postinsts.service | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/meta/recipes-devtools/run-postinsts/run-postinsts/run-postinsts.service b/meta/recipes-devtools/run-postinsts/run-postinsts/run-postinsts.service
index d42addf510ce..7f72f3388afb 100644
--- a/meta/recipes-devtools/run-postinsts/run-postinsts/run-postinsts.service
+++ b/meta/recipes-devtools/run-postinsts/run-postinsts/run-postinsts.service
@@ -8,7 +8,7 @@ Before=sysinit.target
Type=oneshot
ExecStart=#SBINDIR#/run-postinsts
ExecStartPost=#BASE_BINDIR#/systemctl --no-reload disable run-postinsts.service
-RemainAfterExit=No
+RemainAfterExit=yes
TimeoutSec=0
[Install]
--
2.17.1
next prev parent reply other threads:[~2020-04-22 13:52 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-04-22 13:52 [OE-Core][PATCH 1/6] openssh: Move sshdgenkeys.service to sshd.socket Alex Kiernan
2020-04-22 13:52 ` [OE-Core][PATCH 2/6] rngd-tools: Install as part of sysinit.target Alex Kiernan
2020-04-22 13:52 ` [OE-Core][PATCH 3/6] rng-tools: Restrict rngd.service Alex Kiernan
2020-04-22 13:52 ` Alex Kiernan [this message]
2020-04-22 13:52 ` [OE-Core][PATCH 5/6] systemd: Drop RDEPEND on systemd-compat-units when sysvinit is disabled Alex Kiernan
2020-04-22 13:52 ` [OE-Core][PATCH 6/6] systemd: upgrade v244.3 -> v245.5 Alex Kiernan
2020-04-22 18:48 ` [OE-Core][PATCH 1/6] openssh: Move sshdgenkeys.service to sshd.socket Andre McCurdy
2020-04-23 8:30 ` Alex Kiernan
2020-04-23 17:14 ` Andre McCurdy
2020-04-25 1:43 ` Khem Raj
2020-04-25 10:44 ` Alex Kiernan
2020-04-25 15:12 ` Khem Raj
[not found] ` <160828A1D0F44509.29104@lists.openembedded.org>
2020-04-23 8:38 ` [OE-Core][PATCH 3/6] rng-tools: Restrict rngd.service Alex Kiernan
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20200422135211.121292-4-alex.kiernan@gmail.com \
--to=alex.kiernan@gmail.com \
--cc=openembedded-core@lists.openembedded.org \
--cc=ross.burton@intel.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox