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

Information about resolving sysroot error #152

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

lstefan1520
Copy link

From testing on a few dedicated server providers, I ran into an issue with GRUB not working properly after the upgrade to RockyLinux 8.5
Example error message:
image
image

This is fixable by remounting all drives within a rescue system, then rebuilding grub cfg.

@pajamian
Copy link
Collaborator

pajamian commented Jan 30, 2022

We currently have some similar code in the fix_efi function to fix up EFI boot. Should we be doing this for legacy boot as well?

@lstefan1520
Copy link
Author

We currently have some similar code in the fix_efi function to fix up EFI boot. Should we be doing this for legacy boot as well?

Perhaps we should. The servers I was having issues with were booting via legacy BIOS. I'll suggest those modifications perhaps?

@pajamian
Copy link
Collaborator

pajamian commented Jan 30, 2022

Yes, it's unusual for legacy boot to require running grub2-mkconfig, but it shouldn't hurt. Do you want to open a new PR for that? Follow the example of fix_efi and make another function fix_legacy that just does the appropriate command, and call that if EFI boot is not set (from the same place where we call fix_efi).

@komitov
Copy link

komitov commented Feb 18, 2022

I would suggest a cleaner way of dealing with this: #162
Indeed, we do need to run grub2-mkconfig -o /boot/grub2/grub.cfg on a machine, that boots the BIOS way even if it is UEFI capable.

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.

3 participants