-
Notifications
You must be signed in to change notification settings - Fork 97
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
🐛 Create journald persistent path #1324
Conversation
Journald will not save any logs to disk if it finds that there is no /var/log/journal directory created (wut?) so make sure to create the dir during initramfs stage so the logs are persisted across reboots Signed-off-by: Itxaka <itxaka.garcia@spectrocloud.com>
✅ Deploy Preview for kairos-io canceled.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great finding!!!
Codecov Report
📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more @@ Coverage Diff @@
## master #1324 +/- ##
==========================================
- Coverage 22.79% 21.10% -1.69%
==========================================
Files 22 16 -6
Lines 1610 1284 -326
==========================================
- Hits 367 271 -96
+ Misses 1179 962 -217
+ Partials 64 51 -13
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
Good catch! I'm guessing if this instead should be a .keep file in our overlay, but dynamically seems much better, so we detect the OS instead |
Journald will not save any logs to disk if it finds that there is no /var/log/journal directory created (wut?) so make sure to create the dir during initramfs stage so the logs are persisted across reboots
What this PR does / why we need it:
Which issue(s) this PR fixes (optional, in
fixes #<issue number>(, fixes #<issue_number>, ...)
format, will close the issue(s) when PR gets merged):Fixes #597