-
Notifications
You must be signed in to change notification settings - Fork 401
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
fix(dracut-shutdown): add cleanup handler on failure #1689
Conversation
…ter dracut-shutdown-onfailure.service It may happen that dracut-shutdown.service fails, for example on timeout due to very low bandwidth. In such case, for hardening purposes, a new dracut-shutdown-onfailure.service unit doing dracut-shutdown.service cleanup needs to execute first, which will ensure switching root to an incomplete initramfs wont't occur. See related dracut PR #1689 (dracutdevs/dracut#1689). Signed-off-by: Renaud Métrich <[email protected]>
…ter dracut-shutdown-onfailure.service It may happen that dracut-shutdown.service fails, for example on timeout due to very low bandwidth. In such case, for hardening purposes, a new dracut-shutdown-onfailure.service unit doing dracut-shutdown.service cleanup needs to execute first, which will ensure switching root to an incomplete initramfs doesn't occur. See related dracut PR #1689 (dracutdevs/dracut#1689). Signed-off-by: Renaud Métrich <[email protected]>
See also related plymouth PR https://github.com/freedesktop/plymouth/pull/6 |
Serial console output on success (similar to current code):
Serial console output on failure (no jump to initramfs):
|
@rmetrich looking at that bz.rh report I think you should do two things before proceeding further a) try to find what's the actual root cause for the unpacking failing in the first place and b) try to duplicate it on a distribution which has more modern core/baseOS stack since RHEL releases are made out of outdate core/base OS components. |
I agree RHEL is quite behind but IMHO hardening is still valuable. I saw this recently on HP hardware which was sending burst of Ctrl-Alt-Del. There was a PR to harden this which is now upstream (commit #b9ba3c8bb8f0f1328cd1ffaa8dbf64585b28c474). |
With regards to the dvd/ilo issue is not the issue there that the admin has not increased the idle timeout in the firmware? In properly designed OS it needs to be consistent throughout the OS ( which is not the case in Fedora/rhel ) and the relevant hw firmware as well. |
Head branch was pushed to by a user without write access
It may happen that dracut-shutdown.service fails, for example on timeout due to very low bandwidth. In such case, for hardening purposes, a new dracut-shutdown-onfailure.service unit doing dracut-shutdown.service cleanup needs to execute to make sure switching root to an incomplete initramfs won't occur later. See also RHBZ #1924587 (https://bugzilla.redhat.com/show_bug.cgi?id=1924587).
dd77682
to
9eaf80f
Compare
It may happen that
dracut-shutdown.service
fails, for example on timeout due to very low bandwidth.In such case, for hardening purposes, a new
dracut-shutdown-onfailure.service
unit doingdracut-shutdown.service
cleanup needs to execute to make sure switching root to an incomplete initramfs won't occur later.See also RHBZ #1924587.
This pull request changes...
Changes
Checklist
Fixes #