15678 2024-12-03 16:52:54 +0000 Systemd disables creation of /var/log/journal and related directories 2025-02-06 01:22:54 +0000 1 1 7 Build System, Metadata & Runtime OE-Core core 5.0.5 All Multiple RESOLVED FIXED Medium+ minor 5.2 M2 1 Steffen.Kothe changqing.li meta.mr.watcher meta.watcher niko.mauno raj.khem randy.macleod --- --- No (bug/feature does not impact docs) oldest_to_newest 100340 0 5086 Steffen.Kothe 2024-12-03 16:52:54 +0000 Created attachment 5086 Suggestion for systemd upstream fix when bug is confirmed in Yocto 18d46e11d85d ("systemd: fix dead link /var/log/README") Disabled the build time creation of the /var/log/journal directories in favor of keeping a README which is mostly irrelevant for most embedded Linux systems, usually not shipping documentation. Additionally does this "fix" change the behavior of VOLATILE_LOG_DIR, since logs are not persistently stored anymore, because systemd does not recreate the directory structure during boot via systemd-tmpfiles.d. While non-persistent logs are not getting affected by this change does it affect the persistent storing of logs. They are lost, so it can be assumed that the mentioned fixed introduced a bug when VOLATILE_LOG_DIR = "1" is set. Recommendation/counter measure: Revert this commit and drop the suspicious README link from the legacy configuration file via sed script or patch. A patch is attached, which was not sent upstream yet to re-enable. A contribution with a fix is intended to be sent when the bug is confirmed. 100352 1 randy.macleod 2024-12-05 15:37:28 +0000 Sandy/Changqing, Did you or someone else in the WR team change this? Can someone take a look, please. Nico may be involved, I don't have time to check right now, sorry. 100368 2 changqing.li 2024-12-06 04:38:38 +0000 @steffen, @Randy Please refer this V1 patch comments and V5 patch comments: https://patchwork.yoctoproject.org/project/oe-core/patch/20230629101038.3772750-1-changqing.li@windriver.com/ https://patchwork.yoctoproject.org/project/oe-core/patch/20240227051155.2166683-1-changqing.li@windriver.com/ And then finally we have this commit: 18d46e11d85d ("systemd: fix dead link /var/log/README") The attached patch is the same as the V1 patch. 100391 3 Steffen.Kothe 2024-12-09 17:49:37 +0000 Circumventing this issue for creating the /var/log/journal can be done by configuring journald.conf to make the storage persistent. See Options, "Storage" in https://man7.org/linux/man-pages/man5/journald.conf.5.html Wouldn't it be an idea then to couple VOLATILE_LOG_DIR to this configuration additionally to the V5 patch posted. This is exactly what breaks here due to a README which seems to be part of the doc package anyway or am I wrong? Happy to contribute here. Meanwhile - create-log-dirs also affects the journald companions systemd-journal-remote and gatewayd. This directory is also created by this flag during built time. 100416 4 changqing.li 2024-12-12 10:23:04 +0000 There is behavior change after disabling create-log-dirs: when create-log-dirs is enabled, /var/log/journal will be created, and since "Storage" in journald.conf is default "Auto", when it defects exist of "/var/log/journal", then log will be storge under /var/log/journal. And this is recipe level. And if /var/log/journal is really persistent, it is decided by the distro level config, now it is named as "FILESYSTEM_PERMS_TABLES" now. user only need to change FILESYSTEM_PERMS_TABLES to make systemd log persistent after create-log-dirs is disabled, /var/log/journal is not created, and since "Storage" in journald.conf is default "Auto", since /var/log/journal not exist, /run/log/journal will be used to store log. which is not persistent. if user want persistent log, they need to change "Storage" to "persistent" in journald.conf, then /var/log/journal will be created to store log. Then distro level config "FILESYSTEM_PERMS_TABLES" will decide if it is a really persistent. This cause user have to do extra change of journald.conf to make log persistent. and maybe we should make the behavior back? For the README issue, since this README is not very useful, maybe we can just don't create it, Attached the RFC patch. @Richard, @Khem, what's your opinion? 100417 5 5087 changqing.li 2024-12-12 10:24:05 +0000 Created attachment 5087 enable create-log-dirs 100419 6 Steffen.Kothe 2024-12-12 12:00:08 +0000 There was a discussion about the systemd-doc packages which suggested to keep the README there. Having a link for no reason makes no sense, so dropping it seems to be okay. This restores the creation of the directories anyway and even leaves, if required, the generation of journald associated packages and its users intact. 100849 7 changqing.li 2025-02-06 01:17:44 +0000 enable create-log-dirs in this commit: https://git.openembedded.org/openembedded-core/commit/?id=f82d9c997ba8cc23b472d44a43489c597bf452af 100850 8 changqing.li 2025-02-06 01:22:54 +0000 enable create-log-dirs in this commit: https://git.openembedded.org/openembedded-core/commit/?id=f82d9c997ba8cc23b472d44a43489c597bf452af 5086 2024-12-03 16:52:54 +0000 2024-12-03 16:52:54 +0000 Suggestion for systemd upstream fix when bug is confirmed in Yocto 0001-tmpfiles.d-Drop-README-link-from-legacy.conf.patch text/plain 1625 Steffen.Kothe RnJvbSAxN2RlOWU5ZDZjYWU4YTUxYmJkMjg2NGQ5NWJiYzQ0NzQ4YzE1ZjY5IE1vbiBTZXAgMTcg MDA6MDA6MDAgMjAwMQpGcm9tOiBTdGVmZmVuIEtvdGhlIDxzdGVmZmVuLmtvdGhlQGR1YWdvbi5j b20+CkRhdGU6IEZyaSwgMjkgTm92IDIwMjQgMTk6NDQ6MjQgKzAxMDAKU3ViamVjdDogW1BBVENI IDEvMV0gdG1wZmlsZXMuZDogRHJvcCBSRUFETUUgbGluayBmcm9tIGxlZ2FjeS5jb25mCgpGb2xs b3dpbmcgY29tbWl0IGluIHRoZSBvcGVuZW1iZWRkZWQtY29yZSBkZWNsYXJlZCB0aGF0Ci1EY3Jl YXRlLWxvZy1kaXJzIGNhdXNlcyB0cm91YmxlIHdoZW4gWW9jdG8gZGVmaW5lcyAvdmFyL2xvZyBh cyB2b2xhdGlsZQphbmQgbm9uLXZvbGF0aWxlIGVxdWFsbHk6CgogLSBMaW5rOiBnaXQ6Ly9naXQu b3BlbmVtYmVkZGVkLm9yZy9vcGVuZW1iZWRkZWQtY29yZQogLSAxOGQ0NmUxMWQ4NWQgKCJzeXN0 ZW1kOiBmaXggZGVhZCBsaW5rIC92YXIvbG9nL1JFQURNRSIpCgpUaGlzIGNoYW5nZSBjYXVzZXMg dGhhdCBuZWl0aGVyIC92YXIvbG9nL2pvdXJuYWwgbm9yIHRoZSByZXN1bHRpbmcKL3Zhci9sb2cv am91cm5hbC9yZW1vdGUgZGlyZWN0b3J5IGlzIGNyZWF0ZWQuCgpBZm9yZSBtZW50aW9uZWQgZGly ZWN0b3JpZXMgYXJlIGFsc28gbm90IGNyZWF0ZWQgYnkgYW55IHRtcGZpbGVzLmQKbWVjaGFuaXNt LgoKUkVBRE1FIGxpbmsgYWZmZWN0cyB0aGUgZGlyZWN0b3J5IGNyZWF0aW9uIGFuZCBwcmV2ZW50 cyB0aGUgam91cm5hbApkaXJlY3RvcnkgdG8gYmUgY3JlYXRlZC4KClNvIGRyb3AgdGhlIGxpbmsg dG8gdGhlIFJFQURNRSBpbiBmYXZvciBvZiBrZWVwaW5nIC1EY3JlYXRlLWxvZy1kaXJzCnN0YWJs ZSBmb3IgdXNlciBhbmQgZW1iZWRkZWQgb3JpZW50YXRlZCBMaW51eCBtYWNoaW5lcy4KCkRpcmVj dG9yaWVzIG11c3QgYmUgY3JlYXRlZCBhbnkgdGltZSByZWdhcmRsZXNzIG9mIGEgUkVBRE1FIHdo aWNoIGlzCmxvY2F0ZWQgaW4gZG9jX2Rpci4KClVwc3RyZWFtLVN0YXR1czogUGVuZGluZwoKU2ln bmVkLW9mZi1ieTogU3RlZmZlbiBLb3RoZSA8c3RlZmZlbi5rb3RoZUBkdWFnb24uY29tPgotLS0K IHRtcGZpbGVzLmQvbGVnYWN5LmNvbmYuaW4gfCAzIC0tLQogMSBmaWxlIGNoYW5nZWQsIDMgZGVs ZXRpb25zKC0pCgpkaWZmIC0tZ2l0IGEvdG1wZmlsZXMuZC9sZWdhY3kuY29uZi5pbiBiL3RtcGZp bGVzLmQvbGVnYWN5LmNvbmYuaW4KaW5kZXggNGYyYzBkN2M0My4uNjJlMmFlMDk4NiAxMDA2NDQK LS0tIGEvdG1wZmlsZXMuZC9sZWdhY3kuY29uZi5pbgorKysgYi90bXBmaWxlcy5kL2xlZ2FjeS5j b25mLmluCkBAIC0xMiw5ICsxMiw2IEBACiAKIGQgL3J1bi9sb2NrIDA3NTUgcm9vdCByb290IC0K IEwgL3Zhci9sb2NrIC0gLSAtIC0gLi4vcnVuL2xvY2sKLXslIGlmIENSRUFURV9MT0dfRElSUyAl fQotTCAvdmFyL2xvZy9SRUFETUUgLSAtIC0gLSAuLi8uLnt7RE9DX0RJUn19L1JFQURNRS5sb2dz Ci17JSBlbmRpZiAlfQogCiAjIC9ydW4vbG9jay9zdWJzeXMgaXMgdXNlZCBmb3Igc2VyaWFsaXpp bmcgU3lzViBzZXJ2aWNlIGV4ZWN1dGlvbiwgYW5kCiAjIGhlbmNlIHdpdGhvdXQgdXNlIG9uIFN5 c1YtbGVzcyBzeXN0ZW1zLgotLSAKMi4zNC4xCgo= 5087 2024-12-12 10:24:05 +0000 2024-12-12 10:24:05 +0000 enable create-log-dirs 0001-systemd-enable-create-log-dirs.patch text/plain 1647 changqing.li RnJvbSBhNzliYjg1ZjE4NzFmMTc0MjEyOWVjN2RkNTY4YmFjZWY4NjQ5NzUxIE1vbiBTZXAgMTcg MDA6MDA6MDAgMjAwMQpGcm9tOiBDaGFuZ3FpbmcgTGkgPGNoYW5ncWluZy5saUB3aW5kcml2ZXIu Y29tPgpEYXRlOiBUaHUsIDEyIERlYyAyMDI0IDE4OjE2OjU3ICswODAwClN1YmplY3Q6IFtQQVRD SF0gc3lzdGVtZDogZW5hYmxlIGNyZWF0ZS1sb2ctZGlycwoKd2UgZGlzYWJsZSBjcmVhdGUtbG9n LWRpcnMgZm9yIGZpeGluZyBkZWFkIGxpbmsgL3Zhci9sb2cvUkVBRE1FLApidXQgdGhpcyBtYWtl IHN5c3RlbWQgbG9nIGFsd2F5cyBub24gcGVyc2lzdGVudCwgZXZlbnQgd2UgZG9uJ3QKc2V0IHZv bGF0aWxlIGxvZywgdW5sZXNzIHVzZXIgdG8gY2hhbmdlIGpvdXJuYWxkLmNvbmYsIG1ha2UKIlN0 b3JhZ2UiIHRvICJwZXJzaXN0ZW50Ii4KCnNvIHJlZW5hYmxlIGNyZWF0ZS1sb2ctZGlycywgYW5k IHNpbmNlIFJFQURNRSBpcyBub3QgdmVyeSB1c2VyZnVsLApqdXN0IHJlbW92ZSBpdC4KClNpZ25l ZC1vZmYtYnk6IENoYW5ncWluZyBMaSA8Y2hhbmdxaW5nLmxpQHdpbmRyaXZlci5jb20+Ci0tLQog bWV0YS9yZWNpcGVzLWNvcmUvc3lzdGVtZC9zeXN0ZW1kXzI1Ni44LmJiIHwgNSArKysrLQogMSBm aWxlIGNoYW5nZWQsIDQgaW5zZXJ0aW9ucygrKSwgMSBkZWxldGlvbigtKQoKZGlmZiAtLWdpdCBh L21ldGEvcmVjaXBlcy1jb3JlL3N5c3RlbWQvc3lzdGVtZF8yNTYuOC5iYiBiL21ldGEvcmVjaXBl cy1jb3JlL3N5c3RlbWQvc3lzdGVtZF8yNTYuOC5iYgppbmRleCBjMTkxNWQ3ZTk0Li5jZDY3MDdi MTkzIDEwMDY0NAotLS0gYS9tZXRhL3JlY2lwZXMtY29yZS9zeXN0ZW1kL3N5c3RlbWRfMjU2Ljgu YmIKKysrIGIvbWV0YS9yZWNpcGVzLWNvcmUvc3lzdGVtZC9zeXN0ZW1kXzI1Ni44LmJiCkBAIC0y NDUsNyArMjQ1LDYgQEAgRVhUUkFfT0VNRVNPTiArPSAiLURub2JvZHktdXNlcj1ub2JvZHkgXAog ICAgICAgICAgICAgICAgICAgLURzeXN0ZW0tdWlkLW1heD05OTkgXAogICAgICAgICAgICAgICAg ICAgLURzeXN0ZW0tYWxsb2MtZ2lkLW1pbj0xMDEgXAogICAgICAgICAgICAgICAgICAgLURzeXN0 ZW0tZ2lkLW1heD05OTkgXAotICAgICAgICAgICAgICAgICAgLURjcmVhdGUtbG9nLWRpcnM9ZmFs c2UgXAogICAgICAgICAgICAgICAgICAgJHtAYmIudXRpbHMuY29udGFpbnMoJ0RJU1RST19GRUFU VVJFUycsICd6ZXJvY29uZicsICctRGRlZmF1bHQtbWRucz1ubyAtRGRlZmF1bHQtbGxtbnI9bm8n LCAnJywgZCl9IFwKICAgICAgICAgICAgICAgICAgICIKIApAQCAtMzkxLDYgKzM5MCwxMCBAQCBk b19pbnN0YWxsKCkgewogCQkJc2VkIC1pICcvQWx0ZXJuYXRpdmVOYW1lc1BvbGljeT0vcy8kLyBt YWMvJyAke0R9JHtub25hcmNoX2xpYmRpcn0vc3lzdGVtZC9uZXR3b3JrLzk5LWRlZmF1bHQubGlu awogCQlmaQogCWZpCisKKwlpZiBbIC1lICR7RH0ke25vbmFyY2hfbGliZGlyfS90bXBmaWxlcy5k L2xlZ2FjeS5jb25mIF07dGhlbgorCQlzZWQgLWkgLWUgJy9eTCBcL3ZhclwvbG9nXC9SRUFETUUv ZCcgJHtEfSR7bm9uYXJjaF9saWJkaXJ9L3RtcGZpbGVzLmQvbGVnYWN5LmNvbmYKKwlmaQogfQog CiBweXRob24gcG9wdWxhdGVfcGFja2FnZXM6cHJlcGVuZCAoKXsKLS0gCjIuMjUuMQoK