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

install: Enable bootprefix by default #193

Merged
merged 1 commit into from
Nov 10, 2023

Conversation

cgwalters
Copy link
Collaborator

This is the last bit necessary to enable installations to systems without a separate /boot partition (as is true in the default RHEL 9 AMIs).

I was too chicken to enable it by default in ostree long ago; and now here we are. Should definitely enable it in ostree, but let's do it here now.

This is the last bit necessary to enable installations to
systems without a separate `/boot` partition (as is true in
the default RHEL 9 AMIs).

I was too chicken to enable it by default in ostree long ago;
and now here we are.  Should definitely enable it in ostree,
but let's do it here now.

Signed-off-by: Colin Walters <walters@verbum.org>
Copy link
Member

@vrothberg vrothberg left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Member

@vrothberg vrothberg left a comment

Choose a reason for hiding this comment

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

Any chance the property can be tested?

@cgwalters cgwalters merged commit b57ddbd into containers:main Nov 10, 2023
9 checks passed
@cgwalters
Copy link
Collaborator Author

Yep will followup, thanks

cgwalters added a commit to cgwalters/bootc that referenced this pull request Nov 12, 2023
cgwalters added a commit to cgwalters/bootc that referenced this pull request Nov 12, 2023
Followup to containers#193

Signed-off-by: Colin Walters <walters@verbum.org>
@cgwalters
Copy link
Collaborator Author

@vrothberg promised followup in #204

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.

None yet

2 participants