-
Notifications
You must be signed in to change notification settings - Fork 59
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
Configure kdump to exclude some coreos modules in generated kdump initramfs #1832
Comments
I think the biggest question in my mind would be whether we'd want to configure this in kdump itself (i.e. upstream kdump and delivered along with the kdump RPM) or separately in our overlay. |
Maybe related : rhkdump/kdump-utils#11 (comment) |
This makes sense to me. Though it feels like it should live in f-c-c instead of kdump? We could probably even dynamically generate the exclusion config at build time so we don't have to maintain separate lists. Hmm, do we even need any of the CoreOS dracut modules in kdump? That'd be easy (e.g. just collect all modules with |
If you look at #1837, it looks like they want to do the reverse, in that we would have to opt-in the coreos modules that we want in the kdump initramfs if my understanding is correct. We mostly need the ostree one when we are dumping to the local system from memory, and maybe afterburn for networking? Otherwise I don't think we need anything else. |
We briefly talked about that issue in today's community meeting and it looks like #1837 should be a good next step. |
Hi team, I noticed that ignition is not listed in dracut.conf.d, it is added as dependency or something else. The real problem is that the ignition related modules are too big, so I suggest to exclude the unnecessary modules when building initramfs. |
@licliu it should be fixed with coreos/fedora-coreos-config#3289 if I understand correctly ? |
@jbtrystram I used to think the same, but testing revealed that Ignition is unconditionally included in the initramfs, even though it is not specified in any
After removing the ignition module, the size of initramfs will be significantly reduced, from 66M to 43M. PS: before coreos/fedora-coreos-config#3289, the initramfs in my test machine is 183M. |
We disable host-only for our initramfs builds: |
Overall, while it should work to build using hostonly for kdump right now, this is not the path we should take in the future, as we ideally should build the kdump initramfs image in the container and not on the host. In the mean time, we can probably add what's needed to support the host-only mode to the ignition modules. I don't know what's needed exactly. |
From https://man7.org/linux/man-pages/man7/dracut.modules.7.html:
|
Probably needs changes to https://github.com/coreos/fedora-coreos-config/blob/testing-devel/overlay.d/05core/usr/lib/dracut/modules.d/35coreos-ignition/module-setup.sh or https://github.com/coreos/ignition/blob/main/dracut/30ignition/module-setup.sh as the one above is a depends. |
There are some low memory cases where kdump is failing to start.
The initramfs generated for kdump includes all the modules used for CoreOS, including Ignition, which make it larger than needed (i.e. Ignition shouldn't and won't run on the kdump boot). So there is opportunity for us to reduce the size (and thus memory requirements) through configuration.
This request originated in https://issues.redhat.com/browse/OCPBUGS-44368
The text was updated successfully, but these errors were encountered: