coreos / fedora-coreos-tracker

Issue tracker for Fedora CoreOS
https://fedoraproject.org/coreos/
263 stars 59 forks source link

kdump to an NFS share require additional configuration (was: Consider adding NFS module to initramfs) #1729

Open jbtrystram opened 4 months ago

jbtrystram commented 4 months ago

Currently, the nfs kernel module is excluded from initramfs as we don't support root on NFS. However, this prevents kdump to use NFS as a target for logs.

So maybe we could add back that module, or find a way to enable it only in the kdump case ?

See https://issues.redhat.com/browse/OCPBUGS-31755

travier commented 4 months ago

While looking at something else, I found https://github.com/coreos/fedora-coreos-config/blob/testing-devel/overlay.d/05core/usr/lib/dracut/dracut.conf.d/coreos-omits.conf

jbtrystram commented 4 months ago

yes we explicitely omit it there that's why I opened this

gursewak1997 commented 3 months ago

Discussed in the community meeting This issue should be discussed/fixed with upstream kdump maintainers i.e. i.e. the fix here isn't in our stack, but upstream.

travier commented 3 months ago

We should document how to workaround this issue for kdump.

It should be possible to force the inclusion of the nfs module in the kdump initramfs by passing the --force-add nfs argument to the dracut call via the dracut_args config option in kdump.conf.

jbtrystram commented 3 months ago

@travier another easy way is just to add extra_modules nfs to kdump.conf

Surprisingly dracut_args does not work while extra-modules does. See https://github.com/rhkdump/kdump-utils/issues/11#issuecomment-2199689962

travier commented 3 months ago

We should document how to workaround this issue for kdump.

It should be possible to force the inclusion of the nfs module in the kdump initramfs by passing the --force-add nfs argument to the dracut call via the dracut_args config option in kdump.conf.

Apparently this does not work and is tracked in https://issues.redhat.com/browse/RHEL-26114

travier commented 3 months ago

We should add a test for this case.

jbtrystram commented 3 months ago

https://github.com/openshift/openshift-docs/pull/78300

travier commented 2 weeks ago

Might be needed related as well: https://gitlab.com/fedora/bootc/base-images/-/merge_requests/32