Skip to content

init: don't fail if a fs is already mounted#647

Merged
slp merged 1 commit intocontainers:mainfrom
slp:init-allow-reuse-fs
Apr 24, 2026
Merged

init: don't fail if a fs is already mounted#647
slp merged 1 commit intocontainers:mainfrom
slp:init-allow-reuse-fs

Conversation

@slp
Copy link
Copy Markdown
Collaborator

@slp slp commented Apr 24, 2026

This allows us to use init.krun while booting from an initramfs.

Comment thread init/init.c
This allows us to use init.krun while booting from an initramfs.

Signed-off-by: Sergio Lopez <slp@redhat.com>
@slp slp force-pushed the init-allow-reuse-fs branch from 45fe1b9 to 74aa373 Compare April 24, 2026 11:38
@dorindabassey
Copy link
Copy Markdown
Collaborator

/gemini review

Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request modifies the mount_filesystems function in init/init.c to ignore EBUSY errors, which prevents the boot process from failing if core filesystems like /proc, /sys, and /dev/pts are already mounted. A review comment suggests that this logic should also be applied to other mount calls within the same file, specifically in chroot_luks and sev_get_luks_passphrase, to ensure consistent behavior and avoid boot failures in certain environments.

Comment thread init/init.c
Copy link
Copy Markdown
Collaborator

@mtjhrc mtjhrc left a comment

Choose a reason for hiding this comment

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

LGTM

@slp slp merged commit 0fb4e84 into containers:main Apr 24, 2026
14 checks passed
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.

3 participants