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

sdboot: forward port content from PR#5297 #5458

Merged
merged 2 commits into from
Feb 8, 2024

Conversation

KKoukiou
Copy link
Contributor

@KKoukiou KKoukiou commented Feb 6, 2024

No description provided.

Live images are already installed with grub2 and cannot be changed on
the fly to systemd-boot (well at least without a lot of extra
work). In case the user passed one of the systemd-boot options to
anaconda, we should be a bit more graceful than failing to run the
updateloader entries script as currently happens.

This change, when run on non webUI installs results in a popup
notifying the user there is an error in the bootloader configuration
and asking if they would like to ignore it.

Resolves: rhbz#2234638
Signed-off-by: Jeremy Linton <jeremy.linton@arm.com>
@pep8speaks
Copy link

pep8speaks commented Feb 6, 2024

Hello @KKoukiou! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

There are currently no PEP 8 issues detected in this Pull Request. Cheers! 🍻

Comment last updated at 2024-02-07 20:27:31 UTC

@github-actions github-actions bot added the f40 label Feb 6, 2024
The installer is not able to verify requirements of the selected bootloader with
other types of payload, like live images and rpmostree installations. Let's
allow systemd-boot only for package installations for now and deal with other
use cases later.

Related: rhbz#2234638
@jkonecny12
Copy link
Member

I fixed the pep8 issue reported here. Hopefully you don't mind.

Copy link
Member

@jkonecny12 jkonecny12 left a comment

Choose a reason for hiding this comment

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

Looks great to me. It even looks great on WebUI.

sdboot-test

@jkonecny12
Copy link
Member

/kickstart-test --testtype smoke

@KKoukiou
Copy link
Contributor Author

KKoukiou commented Feb 8, 2024

Looks great to me. It even looks great on WebUI.

sdboot-test

Hmm however, i see the form is still active and the user can seemingly continue? Or it's not the case.

@jkonecny12
Copy link
Member

It works like this:

  • you enter this spoke
  • press Next
  • the warning message show up
  • pressing Next multiple times won't do anything

The behavior is not great but given to the fact that users have to explicitly change kernel boot arguments or launch Anaconda with a specific --sdboot parameter makes this pretty good enough solution to me.

We can disable the Next button but the output is still the same. Users have to reboot the installation to get rid of this parameter.

@KKoukiou KKoukiou merged commit 52c707a into rhinstaller:master Feb 8, 2024
17 checks passed
@KKoukiou KKoukiou deleted the sdboot-cherry-pickpr-5297 branch February 8, 2024 11:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging this pull request may close these issues.

5 participants