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
[Storage] add btrfs_compression option (#1928857) #3177
[Storage] add btrfs_compression option (#1928857) #3177
Conversation
See storaged-project/blivet#932 for the accompanying Blivet PR With the Blivet PR alone, installing on Fedora 34 results in an uncompressed Btrfs filesystem as before. With a patched Anaconda adding this PR too, the filesystem is compressed with zstd:1 as per |
/test |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Otherwise, it looks good to me. Thanks!
Tested locally with storaged-project/blivet#932 (comment). Without the fix, it fails with a traceback. |
This is a new option to allow specifying the compression algorithm and level for Btrfs filesystems. If set, it will be passed to Blivet and applied when mounting Btrfs filesystems. Default this to None, but set it to "zstd:1" for Fedora. Signed-off-by: Michel Alexandre Salim <michel@michel-slm.name>
c200150
to
15e06b6
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It looks great. Thank you!
Blivet 3.3.3 is out. |
It's working! Fedora-Workstation-Live-x86_64-34-20210219.n.0.iso
|
This is a new option to allow specifying the compression algorithm and
level for Btrfs filesystems. If set, it will be passed to Blivet and
applied when mounting Btrfs filesystems.
Default this to None, but set it to "zstd:1" for Fedora.
Signed-off-by: Michel Alexandre Salim michel@michel-slm.name
This change is