Skip to content

Return the built filesystem from generate_boot_image - #376

Merged
Johan-Liebert1 merged 1 commit into
composefs:mainfrom
Johan-Liebert1:api-update
Aug 11, 2026
Merged

Return the built filesystem from generate_boot_image#376
Johan-Liebert1 merged 1 commit into
composefs:mainfrom
Johan-Liebert1:api-update

Conversation

@Johan-Liebert1

Copy link
Copy Markdown
Collaborator

In bootc we call this function to generate the boot EROFS image, then we again perform a very expensive operation of re-creating the filesystem.

generate_boot_image already creates a filesystem, so simply return that

giuseppe
giuseppe previously approved these changes Aug 11, 2026

@giuseppe giuseppe left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

In bootc we call this function to generate the boot EROFS image, then we
again perform a very expensive operation of re-creating the filesystem.

`generate_boot_image` already creates a filesystem, so simply return
that

Signed-off-by: Pragyan Poudyal <pragyanpoudyal41999@gmail.com>
@Johan-Liebert1

Copy link
Copy Markdown
Collaborator Author

Fixed a clippy warning

@Johan-Liebert1
Johan-Liebert1 added this pull request to the merge queue Aug 11, 2026
Merged via the queue into composefs:main with commit ad778f2 Aug 11, 2026
19 checks passed
Johan-Liebert1 added a commit to Johan-Liebert1/composefs-rs that referenced this pull request Aug 12, 2026
In composefs#376, I had forgotten
that we transform the filesystem (which masks /boot) before creating the
EROFS. Making off /boot is the opposite of what we need in bootc since
bootc searches for boot entries in /boot

Signed-off-by: Pragyan Poudyal <pragyanpoudyal41999@gmail.com>
Johan-Liebert1 added a commit to Johan-Liebert1/composefs-rs that referenced this pull request Aug 12, 2026
In composefs#376, I had forgotten
that we transform the filesystem (which masks /boot) before creating the
EROFS. Making off /boot is the opposite of what we need in bootc since
bootc searches for boot entries in /boot

Define a function `generate_boot_image_get_fs` which does the exact same
operations as `generate_boot_image`, but returns the untransformed
filesystem to the caller

Signed-off-by: Pragyan Poudyal <pragyanpoudyal41999@gmail.com>
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