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

Don't install BIOS GRUB in sd-boot images #167

Merged
merged 3 commits into from
Aug 22, 2024

Conversation

wjt
Copy link
Member

@wjt wjt commented Aug 21, 2024

systemd-boot is EFI-only. If we use systemd-boot on an image, we
actively do not want the system to be BIOS-bootable. (And the installed
BIOS GRUB doesn't actually work on these images anyway.)

On sd-boot images, don't install any BIOS bootloader and don't create a BIOS Boot partition.

In addition, drop code for the long-unused cros partition layout.

https://phabricator.endlessm.com/T34775

In essence this reverts commit 074ade3.

The device this was intended to support never went into production. In
addition, config/schema.ini rejects this value for `[image]
partition_table`!

https://phabricator.endlessm.com/T34775
systemd-boot is EFI-only. If we use systemd-boot on an image, we
actively do not want the system to be BIOS-bootable. (And the installed
BIOS GRUB doesn't actually work on these images anyway.)

https://phabricator.endlessm.com/T34775
This partition was never useful and is now unused.

We will find out which of our tools assume a 3-partition disk image...

https://phabricator.endlessm.com/T34775
@wjt
Copy link
Member Author

wjt commented Aug 21, 2024

Scratch build running at https://ci.endlessos.org/job/image-build-amd64/30438

@wjt
Copy link
Member Author

wjt commented Aug 22, 2024

Tested that image in 2 ways:

  1. Booted qcow2 image in Boxes. It works correctly and the root filesystem is correctly resized to fill the disk.
  2. Installed the .img.xz on a sacrificial laptop using eosinstaller. eos-installer correctly recognised that it is an Endless OS disk even though the BIOS Boot partition is missing. This is expected because we dropped that requirement in 2020 for GNOME OS – the installer now only checks that the disk has at least 2 partitions: ESP and OS. The resulting system booted correctly and the root filesystem was correctly expanded to fill the disk.

@wjt wjt marked this pull request as ready for review August 22, 2024 10:49
Copy link
Member

@dbnicholson dbnicholson left a comment

Choose a reason for hiding this comment

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

Makes sense, but it obviously instills a bit of fear. Godspeed!

@dbnicholson dbnicholson merged commit ff6c562 into master Aug 22, 2024
2 checks passed
@dbnicholson dbnicholson deleted the T34775-no-grub-in-sdboot-images branch August 22, 2024 18:00
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.

2 participants