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

RHEL 9: bootupd update firmware #5760

Draft
wants to merge 2 commits into
base: rhel-9
Choose a base branch
from

Conversation

jkonecny12
Copy link
Member

@jkonecny12 jkonecny12 commented Jul 13, 2024

Enable installation of the EFI entry from the bootupd project.

Backport of the #5508.

Resolves: RHEL-40897

  • Blocked on RHEL-40897 still being in PLANNING

Copy link
Contributor

@M4rtinK M4rtinK left a 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.

This is required to write an entry to the EFI boot manager,
which we ought to do (anaconda does it when installing the
bootloader itself). Without this, boot of the installed system
will only work if it's configured to try and boot from the
hard disk using the fallback path.

Signed-off-by: Adam Williamson <[email protected]>
(cherry picked from commit 0b2fc13)

Resolves: RHEL-40897
@jkonecny12 jkonecny12 force-pushed the rhel-9-bootupd-update-firmware branch from da16a0d to 223b0e6 Compare July 24, 2024 12:06
@jkonecny12
Copy link
Member Author

/kickstart-test --kstest-pr 1260 rpm-ostree-container-bootc

If leavebootorder was specified through kickstart or kernel boot
arguments we won't pass `--update-firmware` to bootupdctl. That will
avoid creation of the UEFI entry for the bootloader and give people
possibility for additional tweaking or debugging.

This was requested by bootloader developers.

Suggested-by: Timothée Ravier <[email protected]>
(cherry picked from commit 28d1aed)

Resolves: RHEL-40897
@jkonecny12 jkonecny12 force-pushed the rhel-9-bootupd-update-firmware branch from 223b0e6 to 7394e1f Compare July 24, 2024 13:17
@jkonecny12
Copy link
Member Author

UPDATED:

  • Fix a small issue in tests.

@jkonecny12
Copy link
Member Author

Let's enable this test for RHEL when this will be ready for merge:
rhinstaller/kickstart-tests#1260

@KKoukiou KKoukiou added the blocked Don't merge this pull request! label Aug 15, 2024
Copy link

This PR is stale because it has been open 60 days with no activity.
Remove stale label or comment or this will be closed in 30 days.

@github-actions github-actions bot added the stale label Oct 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
blocked Don't merge this pull request! rhel-9 stale
Development

Successfully merging this pull request may close these issues.

4 participants