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

Use bootupd and remove ostree-grub2 (was: All deployments are shown twice in grub) #120

Open
YaLTeR opened this issue Jan 5, 2021 · 70 comments
Assignees
Labels
enhancement New feature or request f41 Related to Fedora 41 fedora-change Needs a Fedora Change kinoite Also affect Fedora Kinoite rawhide

Comments

@YaLTeR
Copy link

YaLTeR commented Jan 5, 2021

boot-entry-issue

F33, ThinkPad T495s

@felipeborges
Copy link

I can reproduce this too.

@jlebon
Copy link
Member

jlebon commented Jan 5, 2021

@YaLTeR
Copy link
Author

YaLTeR commented Jan 5, 2021

See https://fedoraproject.org/wiki/Common_F31_bugs#On_Fedora_Silverblue.2FIoT.2C_the_GRUB_menu_shows_duplicate_entries (I've added an "Update:" section there).

So... should I migrate to BLS? If I should then maybe it should rather be done by default in Silverblue?

@jlebon
Copy link
Member

jlebon commented Jan 5, 2021

So... should I migrate to BLS? If I should then maybe it should rather be done by default in Silverblue?

If you can, yes. I'm not sure if it's worth forcing a migration at this point because of the potential risks (see e.g. https://discussion.fedoraproject.org/t/boot-entries-gone-after-upgrade/8026). And as mentioned in the wiki, the migration script only works on EFI because it might not be safe to do it automatically on BIOS (see ostreedev/ostree#2044 (comment)).

@Jmennius
Copy link

Jmennius commented Aug 21, 2021

What about systems that did not upgrade to F31 (clean install F31+)?
I did not find any place that creates /boot/grub2/.grub2-blscfg-supported for new installations.
I have BLS enabled (Silverblue) but there is no such file (probably this system started with F31).

I am also concerned that .grub2-switch-to-blscfg does not work well with https://fedoraproject.org/wiki/Changes/UnifyGrubConfig.

cc @jlebon @martinezjavier

@dustymabe
Copy link

I'm seeing this on a fresh install of F35 Silverblue. I guess we need to track down what used to create the /boot/grub2/.grub2-blscfg-supported and see why it's not getting created any longer.

@jlebon
Copy link
Member

jlebon commented Nov 24, 2021

I'm seeing this on a fresh install of F35 Silverblue. I guess we need to track down what used to create the /boot/grub2/.grub2-blscfg-supported and see why it's not getting created any longer.

I think... nothing ever actually created that for new installs. The idea we discussed was to have Anaconda do it, but it looks like we never actually followed through with it.

The other option is to just assume GRUB is new enough, but that can break people upgrading from an old install with old GRUB. The failure mode here is particularly bad: they'll get no boot entries.

Maybe we just need to be really loud about it and say that you need to make sure that your GRUB is up to date before $date using either the migration script or bootupd, and then we rip out this check.

@jlebon
Copy link
Member

jlebon commented Nov 24, 2021

Maybe we just need to be really loud about it and say that you need to make sure that your GRUB is up to date before $date using either the migration script or bootupd, and then we rip out this check.

A prerequisite for that is of course to ship bootupd in FSB, which we currently don't.

@jlebon
Copy link
Member

jlebon commented Nov 25, 2021

Yeah, I think bootupd is probably the cleanest path forward for this and for possible future issues like it. So my strawman is something like:

  • add bootupd in FSB
  • teach Anaconda to call out to bootupctl
  • teach 15_ostree to read bootupd metadata to know if GRUB supports blscfg (but really, if bootupd metadata exists, it surely does, so this might just be test -f /boot/.bootupd-state.json)
  • tell upgrading users that this bug is fixed in the latest bootloader, and they can update using bootupctl adopt-and-update

@cgwalters @martinezjavier WDYT?

@travier
Copy link
Member

travier commented Nov 26, 2021

This might need a Fedora Change for F36 (Silverblue & Kinoite).

@martinezjavier
Copy link

@cgwalters @martinezjavier WDYT?

This makes sense to me. In fact, it would be great if even the non-ostree Fedora variants could adopt bootupd.

@travier
Copy link
Member

travier commented Dec 8, 2021

The change deadline is mid January if we think this is a self contained change: https://fedorapeople.org/groups/schedule/f-36/f-36-key-tasks.html

@ormandj
Copy link

ormandj commented Dec 30, 2021

This would be quite a nice change - I was surprised to see the duplicate entries on a clean 35 Silverblue installation, and ran across this issue while researching resolution.

@travier
Copy link
Member

travier commented Jan 25, 2022

How do all of you feel about trying to make that happen for F37 and maybe opt-in for F36 (ship bootupd but not enabled by default)? I can start the change request.

@tpopela
Copy link
Contributor

tpopela commented Jan 25, 2022

It was great to see @martinezjavier 's option and if he's in favor, then I'm as well. Thank you @travier for looking into this!

@jlebon
Copy link
Member

jlebon commented Jan 25, 2022

How do all of you feel about trying to make that happen for F37 and maybe opt-in for F36 (ship bootupd but not enabled by default)? I can start the change request.

SGTM, thanks for picking this up!

@travier travier added enhancement New feature or request help wanted labels Jun 8, 2022
@tpopela
Copy link
Contributor

tpopela commented Jun 8, 2022

It would be good to tackle this after the recent round of the grub2 CVEs. We might also need to fix coreos/bootupd#108 . But it looks like there was some work done by @cgwalters , but might be not finished - what's the current state of it Colin?

@travier travier changed the title All deployments are shown twice in grub Use bootupd (was: All deployments are shown twice in grub) Jun 8, 2022
@koko-ng
Copy link

koko-ng commented Jul 7, 2022

I ran into this problem while debugging why firmware updates weren't applied. If anyone runs into the same issue, before there is a cleaner fix it's possible to update shim like this:

sudo rpm-ostree usroverlay 
wget https://kojipkgs.fedoraproject.org//packages/shim/15.6/1/x86_64/shim-x64-15.6-1.x86_64.rpm
sudo rpm -i  --reinstall shim-x64-15.6-1.x86_64.rpm

@dustymabe
Copy link

I'm not sure if rpm-ostree usroverlay is what you want:

       usroverlay
           Mount a writable overlay filesystem on /usr which is
           active only for the remainder of the system boot. This
           is intended for development, testing, and debugging.
           Changes will not persist across upgrades, or rebooting
           in general.

@koko-ng
Copy link

koko-ng commented Jul 7, 2022

It's just so that rpm can write to the database, of course it will be discarded on next reboot, but the changes to /bin/efi will effectively be kept as they are not managed by ostree and as rpm reinstall the same version of the package, the content of the persistent database (the one managed by rpm-ostree) is now coherent with the content of /bin/efi.

@travier
Copy link
Member

travier commented Aug 21, 2022

With https://pagure.io/workstation-ostree-config/pull-request/288, we now have bootupd in Rawhide but not enabled by default.

Let's track what's missing here and make a Change request for F38.

For reference, the commit message from Jonathan:

We need to make it easier to update the bootloader on these variants
because unlike on traditional systems, it's not updated automatically
with the rest of the system. Add bootupd for that.

This would allow fixing issues like:
- https://github.com/coreos/rpm-ostree/issues/3715
- https://github.com/fedora-silverblue/issue-tracker/issues/120#issuecomment-979268679

It won't be enabled by default and as mentioned in that comment requires
work in Anaconda to be seamless. But at least with this users should be
able to adopt and update:

https://github.com/coreos/bootupd/blob/main/README-design.md

See also the tracker issue where we did this for Fedora CoreOS:

https://github.com/coreos/fedora-coreos-tracker/issues/510

@travier travier removed the f37 Related to Fedora 37 label Aug 21, 2022
@travier travier removed the f39 Related to Fedora 39 label May 15, 2023
@travier travier added rawhide kinoite Also affect Fedora Kinoite f39 Related to Fedora 39 labels Jul 23, 2023
@travier
Copy link
Member

travier commented Jul 26, 2023

I've merged in Rawhide: https://pagure.io/workstation-ostree-config/pull-request/330

My test images (https://gitlab.com/fedora/ostree/ci-test) also include this already.

@Verhoeckx
Copy link

I've merged in Rawhide: https://pagure.io/workstation-ostree-config/pull-request/330

My test images (https://gitlab.com/fedora/ostree/ci-test) also include this already.

Will this be available in Fedora 39?

The solution in the above comment, didn't work for me. Will your merge request solve the problem?

@Verhoeckx
Copy link

@yann-soubeyrand

I executed the three lines that you mention in the above comment and when I now execute (after a reboot) rpm-ostree upgrade, I get the following error: error: loading sysroot: opendirat: Permission denied. Do I need the execute some other command after the three lines you mention (to close the overlay)?

@juhp
Copy link

juhp commented Aug 19, 2023

Is this still in current Silverblue/Kinoite? I am not seeing it in either 39 or Rawhide, but if so what to look for?

@travier
Copy link
Member

travier commented Aug 19, 2023

This is still planned to be included in F39 but not there yet due to https://pagure.io/workstation-ostree-config/pull-request/330#comment-190406 (the PR has been reverted).

The test images (gitlab.com/fedora/ostree/ci-test) include this already. The Universal Blue images should as well.

@travier
Copy link
Member

travier commented Sep 10, 2023

Pushed back to Fedora 40. Might come as an update to F39 if we backport the support.

@travier
Copy link
Member

travier commented Nov 30, 2023

bootupd support in Anaconda got merged 🎉

@0rzech
Copy link

0rzech commented Dec 17, 2023

I executed the three lines that you mention in the above comment and when I now execute (after a reboot) rpm-ostree upgrade, I get the following error: error: loading sysroot: opendirat: Permission denied. Do I need the execute some other command after the three lines you mention (to close the overlay)?

@Verhoeckx Did you find any solution to this? I've just stumbled upon this error too.

@travier Is this a problem? While I got the same error, rpm-ostree status did display new deployment as usual. If it is a problem, how to fix these permissions?

travier added a commit to travier/ostree that referenced this issue Jan 31, 2024
With the new bootupd installation path in Anaconda, the
`/etc/default/grub` config file is not written anymore as we are only
using BLS configs with new enough bootloaders.

We thus don't need to generate (duplicated) legacy boot entries.

We still need to keep this logic in place in Atomic Desktops
(Silverblue, etc.) until we've actually landed bootupd there and forced
a bootloader update for everybody.

See: fedora-silverblue/issue-tracker#530
See: fedora-silverblue/issue-tracker#120
See: https://fedoraproject.org/wiki/Changes/FedoraSilverblueBootupd
@travier
Copy link
Member

travier commented Apr 11, 2024

See #543 (comment) for a lightly tested set of commands to update your EFI bootloader until we have full bootupd support in Fedora Atomic Desktops.

@scottwillmoore
Copy link

I'm confused 😕, did bootupd make it into 40, or was it deferred until 41? At least from the threads below it looks like it was pushed back to 41, but then was ready for 40, however I don't see bootupd on my installation?

Under the assumption bootupd is not included with 40, is it possible to still use it?

@davidstrauss
Copy link

I researched that exact question about deferral earlier today. I found at least one comment (I think on Bugzilla) that confirmed deferral to F41. I also noticed that most F40 docs fail to reflect that.

@travier
Copy link
Member

travier commented Apr 29, 2024

I've updated everything to point to F41 as we did not land it in F40. We'll try to backport things to F40 as much as possible.

@travier travier added f41 Related to Fedora 41 and removed f40 Related to Fedora 40 labels Apr 29, 2024
@Malix-off
Copy link

Hello @travier, any update about the backport to F40?

@travier
Copy link
Member

travier commented Jul 10, 2024

We can only consider backporting to F40 after it lands in F41. Status is tracked in https://gitlab.com/fedora/ostree/sig/-/issues/1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request f41 Related to Fedora 41 fedora-change Needs a Fedora Change kinoite Also affect Fedora Kinoite rawhide
Projects
None yet
Development

No branches or pull requests