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
Fixes #36189 - Add AlmaLinux UEFI provisioning support #9648
Fixes #36189 - Add AlmaLinux UEFI provisioning support #9648
Conversation
Can one of the admins verify this patch? |
2 similar comments
Can one of the admins verify this patch? |
Can one of the admins verify this patch? |
Issues: #36189 |
In pxegrub2_chainload.erb various paths are tried, but the /EFI/almalinux/grubx64.efi path is missing. This causes the server to refuse to boot. To make it easier to use the AlmaLinux installation medium is added.
9e7f002
to
dc9c885
Compare
ok to test |
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.
Looks good to me and I kicked of CI. @stejskalleos mind having a look?
Code changes LGTM too, but I am still struggling with the setup of the environment for UEFI & HTTP provisioning. When I'll have it I will test it and merge the PR. |
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.
🍏 LGTM
I haven't tested it because still couldn't manage to find time to configure my setup,
however it looks sensible and does same as #9295, so let's get this in.
Thanks @omenos and everyone! |
@ekohl @stejskalleos @Conan-Kudo Thanks for the reviews and merging! |
This is a bit-for-bit copy of #9295, modified for AlmaLinux.
In pxegrub2_chainload.erb various paths are tried, but the /EFI/almalinux/grubx64.efi path is missing. This causes the server to refuse to boot.
To make it easier to use the AlmaLinux installation medium is added.