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

osbuild: set sysroot.bootprefix true in OSTree config #3744

Merged
merged 1 commit into from
Feb 26, 2024

Conversation

dustymabe
Copy link
Member

This is a second round of f5677a3 (reverted in db7bf08).

Now that coreos-installer in rawhide has
coreos/coreos-installer#1422 we can set sysroot.bootprefix to true again.

We can't do it in the non-OSBuild configs because not all streams have the new coreos-installer yet.

Here is the comment from the original commit:

This setting will make it so that BLS config entries get prepended with /boot. OSTree already places a boot -> . symlink in the root of the boot filesystem prepending with /boot will always just work.

For context see osbuild/osbuild#1566 (comment)

This also allows for dropping one of the upstream OSBuild zipl stage patches.

This is a second round of f5677a3 (reverted in db7bf08).

Now that coreos-installer in rawhide has
coreos/coreos-installer#1422
we can set sysroot.bootprefix to true again.

We can't do it in the non-OSBuild configs because not all streams
have the new coreos-installer yet.

Here is the comment from the original commit:

This setting will make it so that BLS config entries get prepended
with /boot. OSTree already places a boot -> . symlink in the root
of the boot filesystem prepending with /boot will always just work.

For context see osbuild/osbuild#1566 (comment)

This also allows for dropping one of the upstream OSBuild zipl stage
patches.
@dustymabe dustymabe merged commit a7d4312 into coreos:main Feb 26, 2024
5 checks passed
@dustymabe dustymabe deleted the dusty-osbuild branch February 26, 2024 17:33
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