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

Mount /boot ro by default #198

Merged
merged 1 commit into from
Feb 15, 2024
Merged

Conversation

cgwalters
Copy link
Contributor

See containers/bootc#294
This is particularly motivated by CentOS/centos-bootc-dev#27 because with that suddenly dnf will appear to start working but trying to do anything involving the kernel (i.e. mutating /boot) will end in sadness, and this puts a stop to that.

(This also relates of course to ye olde #18
where we want the partitioning setup in the default case
to come from the container)

See containers/bootc#294
This is particularly motivated by CentOS/centos-bootc-dev#27
because with that suddenly `dnf` will appear to start working
but trying to do anything involving the kernel (i.e. mutating `/boot`)
will end in sadness, and this puts a stop to that.

(This also relates of course to ye olde osbuild#18
 where we want the partitioning setup in the default case
 to come from the container)

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

@mvo5 mvo5 left a comment

Choose a reason for hiding this comment

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

Thank you

@cgwalters cgwalters added this pull request to the merge queue Feb 15, 2024
Merged via the queue into osbuild:main with commit 711f39b Feb 15, 2024
9 checks passed
@ericcurtin
Copy link

I did read some patches a while pack about adding EROFS support to grub, so you can truely have a ro /boot filesystem with better performance, but they are not ready I believe...

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.

4 participants