-
Notifications
You must be signed in to change notification settings - Fork 42
efi: transfer usr/lib/ostree-boot to usr/lib/efi
#995
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
Conversation
|
Skipping CI for Draft Pull Request. |
76c8909 to
6b7c1d7
Compare
6b7c1d7 to
3c31f36
Compare
feffdab to
1bbe5cd
Compare
1bbe5cd to
6fdfdf5
Compare
When doing tests for coreos#995, find that the package `test-bootupd-payload-1.0-1.x86_64` is parsed to `test-1.0-1` that might be confused, so keep the full name if it is not `grub2` or `shim`
When doing tests for coreos#995, find that the package `test-bootupd-payload-1.0-1.x86_64` is parsed to `test-1.0-1` that might be confused, maybe should keep the full name if it is not `grub2` or `shim`
When doing tests for coreos#995, find that the package `test-bootupd-payload-1.0-1.x86_64` is parsed to `test-1.0-1` that is confused, maybe should keep the full name if it is not `grub2` or `shim`.
When doing tests for coreos#995, find that the package `test-bootupd-payload-1.0-1.x86_64` is parsed to `test-1.0-1` that is confused, maybe should keep the full name if it is not `grub2` or `shim`.
When doing tests for coreos#995, find that the package `test-bootupd-payload-1.0-1.x86_64` is parsed to `test-1.0-1` that is confused, maybe should keep the full name if it is not `grub2` or `shim`.
4818fd4 to
e45e89e
Compare
e45e89e to
ea79d64
Compare
|
|
ea79d64 to
5511c1c
Compare
973d847 to
61f0caa
Compare
cgwalters
left a comment
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 sane to me!
1ef8a9b to
4d2d02f
Compare
- `query_file()` to query the rpm database and get package version - `transfer_ostree_boot_to_usr()` that will copy EFI files from `usr/lib/ostree-boot` to `usr/lib/efi`
`generate_update_metadata()`
Enhance `generate_update_metadata()` with validation to ensure at least two packages are provided, preventing the creation of incomplete or invalid update payloads. See coreos#994
not existing `usr/lib/ostree-boot/efi/EFI`
59de872 to
a3a3493
Compare
|
I think this change was a good idea, but it's caused a multi-day breakage in bootc CI because of the version interlock it creates where older bootupd fails to understand systems whose layout is generated with newer bootupd. As of right now it seems this change is rolled out to most targets except Fedora 42, but it didn't propagate consistently or quickly of course. In retrospect, what we probably should have done is made actually applying this change require an opt-in in the base image (config file say), or We're also working on bootc-dev/bootc#1816 which will obviate most issues of skew, but it's hard to avoid in the general case. With format changes again, the key thing is to decouple understanding the change from application of the format. |
Bootupd is too old, see coreos/bootupd#995 (comment) Signed-off-by: Colin Walters <walters@verbum.org>
The newer bootupd hasn't been rolled out to this image yet, so this should avoid version skew problems. See coreos/bootupd#995 Signed-off-by: Colin Walters <walters@verbum.org>
Bootupd is too old, see coreos/bootupd#995 (comment) Signed-off-by: Colin Walters <walters@verbum.org>
The newer bootupd hasn't been rolled out to this image yet, so this should avoid version skew problems. See coreos/bootupd#995 Signed-off-by: Colin Walters <walters@verbum.org>
Besides this, does it make sense to update bootupd on f42 and c9s? |
|
I think we should keep c9s and c10s in lockstep by default unless there's a rationale against it. For f42...well, I just turned off ci for that in bootc and honestly I don't see a good reason to update it by default, so no strong opinion from me. Again in the end we are likely to fix this more comprehensively in bootc, so it's not urgent. Though also tangential to this I think it's another case where the intersection between bootc and bootupd is fairly coupled and it could still make sense to do a merge. |
Could you help to create such an issue for this?
Agree! |
|
OK let's stop the 0.2.32 release rollout. Sorry I missed this earlier.
Yes, let's do that to make sure we keep existing releases using the current format. We have a change for F44 so we should not change the layout in F43 and C9S at least. Maybe not even C10S but that could be discussed.
+1 |
I think you meant |
efi: transfer
usr/lib/ostree-boottousr/lib/efiThis is required by step2 of #926 (comment), with the change, the files under legacy
/usr/lib/ostree-boot/efi/EFIwill be changed to/usr/lib/efi/<component>/<version>/EFI/.efi: require minimum of 2 components for update metadata generation
Enhance
generate_update_metadata()with validation to ensureat least two packages are provided, preventing the creation of
incomplete or invalid update payloads.
See #994