Skip to content
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(nvmf): require NVMeoF modules (bsc#1230468) (SL-Micro-6.0:Update) #376

Conversation

aafeijoo-suse
Copy link
Collaborator

On some minimal OS images, adding the nvmf dracut module to the initrd does not fail, although the required nvme kernel modules are not shipped with the image. That implies that users didn't see any errors when building the initrd, so they expect booting from NVMe-oF to work, but it obviously fails.

Fixes added:

tbzatek and others added 3 commits September 12, 2024 11:53
When creating initramfs for universal boot image such as an installer,
we can't include any machine-specific IDs. Let's move the check
for /etc/nvme/hostnqn and /etc/nvme/hostid files presence to
the hostonly section to avoid unsatisfied requirements.

Signed-off-by: Tomas Bzatek <[email protected]>
(cherry picked from commit dracut-ng/dracut-ng@54cd647)
Signed-off-by: Martin Wilck <[email protected]>
(cherry picked from commit dracut-ng/dracut-ng@4133270)

bsc#1230468
nvme_rdma was not installed, even if it was required, whereas
nvme_tcp was installed in non-TCP setups. nvme_fabrics will be
pulled in automatically by any transport module and doesn't
need to be installed explicitly.

Fix it.

(cherry picked from commit dracut-ng/dracut-ng@3748ed4)
Copy link
Collaborator

@tblume tblume left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok

@aafeijoo-suse aafeijoo-suse merged commit 06a9ae1 into openSUSE:SL-Micro-6.0_Update Sep 13, 2024
1 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants