[pve-devel] applied: [PATCH installer v2 0/5] fix #5250: add btrfs `compress` mount option support

Thomas Lamprecht t.lamprecht at proxmox.com
Sun Nov 10 19:44:24 CET 2024


Am 13.08.24 um 18:15 schrieb Christoph Heiss:
> Fixes #5250 [0].
> 
> Pretty much as it says on the tin, this adds the `compress` mount option
> for Btrfs much in the same way as the ZFS equivalent.
> 
> W.r.t to the discussion in #5250 - `compress` is used. For a detailed
> explanation, see patch #2.
> 
> [0] https://bugzilla.proxmox.com/show_bug.cgi?id=5250
> 
> History
> =======
> v1: https://lists.proxmox.com/pipermail/pve-devel/2024-May/063845.html
> 
> Notable changes v1 -> v2:
>   * rebased on latest master
>   * squashed auto-installer tests into feature patch
> 
> Diffstat
> ========
> 
> Christoph Heiss (5):
>   fix #5250: install: config: add new `btrfs_opts` with `compress`
>     config option
>   fix #5250: install: write btrfs `compress` option to fstab
>   fix #5250: proxinstall: expose new btrfs `compress` option
>   fix #5250: tui: expose new btrfs `compress` option
>   fix #5250: auto-installer: expose new btrfs `compress` option
> 
>  Proxmox/Install.pm                            | 11 +++++-
>  Proxmox/Install/Config.pm                     | 15 ++++++++
>  proxinstall                                   | 15 ++++++++
>  proxmox-auto-installer/src/answer.rs          |  6 +++-
>  proxmox-auto-installer/src/utils.rs           |  7 +++-
>  .../tests/resources/parse_answer/btrfs.json   | 24 +++++++++++++
>  .../tests/resources/parse_answer/btrfs.toml   | 17 ++++++++++
>  proxmox-installer-common/src/options.rs       | 31 +++++++++++++++++
>  proxmox-installer-common/src/setup.rs         | 21 ++++++++++--
>  proxmox-tui-installer/src/setup.rs            |  2 ++
>  proxmox-tui-installer/src/views/bootdisk.rs   | 34 +++++++++++++------
>  11 files changed, 167 insertions(+), 16 deletions(-)
>  create mode 100644 proxmox-auto-installer/tests/resources/parse_answer/btrfs.json
>  create mode 100644 proxmox-auto-installer/tests/resources/parse_answer/btrfs.toml
> 

applied series with a few merge conflicts resolved, but only the first
one in the first patch for the common's setup module got resolved manually,
maybe recheck to ensure that I did not mess anything up, thanks!




More information about the pve-devel mailing list