Skip to content

elbepack: populate Btrfs images with configured compression - #489

Closed
d4nuu8 wants to merge 1 commit into
Linutronix:masterfrom
d4nuu8:feature/btrfs-mount-options
Closed

d4nuu8 wants to merge 1 commit into
Linutronix:masterfrom
d4nuu8:feature/btrfs-mount-options

Conversation

@d4nuu8

@d4nuu8 d4nuu8 commented Oct 13, 2026 •

Copy link
Copy Markdown
Contributor

Populate Btrfs with mkfs.btrfs -r --compress to avoid exhausting partitions
sized for compressed data.

Derive the compression algorithm and level from mount options while
preserving explicit mkfs overrides. Add regression tests for this behavior.

Comment thread elbepack/fstab.py
@d4nuu8
d4nuu8 force-pushed the feature/btrfs-mount-options branch from fc8f691 to 2b0578e Compare October 13, 2026 13:17
@d4nuu8 d4nuu8 changed the title elbepack: populate Btrfs images with mount options elbepack: populate Btrfs images with configured compression Oct 13, 2026

@t-8ch t-8ch 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.

Nice! A few more comments.

Comment thread elbepack/tests/test_fstab.py Outdated
Comment thread elbepack/fstab.py Outdated
Comment thread elbepack/fstab.py Outdated
Comment thread elbepack/fstab.py Outdated
Populate Btrfs with mkfs.btrfs -r --compress to avoid exhausting partitions
sized for compressed data.

Derive the compression algorithm and level from mount options while
preserving explicit mkfs overrides. Add regression tests for this behavior.

Signed-off-by: Daniel Braunwarth <daniel.braunwarth@kuka.com>
@d4nuu8
d4nuu8 force-pushed the feature/btrfs-mount-options branch from 2b0578e to 6e121fe Compare October 13, 2026 13:53
Comment thread elbepack/fstab.py
@t-8ch

t-8ch commented Oct 13, 2026

Copy link
Copy Markdown
Collaborator

I am testing this now and will push it after.

I am wondering if we should always use the highest compression here automatically.
OTOH users can always use mkfsopts if they want something special.

@d4nuu8

d4nuu8 commented Oct 13, 2026

Copy link
Copy Markdown
Contributor Author

I am wondering if we should always use the highest compression here automatically.

I would stick to the default level.

@t-8ch

t-8ch commented Oct 13, 2026

Copy link
Copy Markdown
Collaborator

I would stick to the default level.

Ack.

@t-8ch

t-8ch commented Oct 13, 2026

Copy link
Copy Markdown
Collaborator

Picked up manually, thanks!

@t-8ch t-8ch closed this Oct 13, 2026
@d4nuu8
d4nuu8 deleted the feature/btrfs-mount-options branch October 14, 2026 07:11
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