[pve-devel] [PATCH installer 3/6] fix #5250: proxinstall: expose new btrfs `compress` option

[Christoph Heiss]

On Mon, May 13, 2024 at 02:13:52PM +0200, Christoph Heiss wrote:
> Signed-off-by: Christoph Heiss <c.heiss at proxmox.com>
> ---
>  Proxmox/Install.pm |  7 +++++--
>  proxinstall        | 15 +++++++++++++++
>  2 files changed, 20 insertions(+), 2 deletions(-)
>
> diff --git a/Proxmox/Install.pm b/Proxmox/Install.pm
> index f3bc5aa..60f38e5 100644
> --- a/Proxmox/Install.pm
> +++ b/Proxmox/Install.pm
> @@ -1014,8 +1014,11 @@ sub extract_data {
>  	    my $btrfs_opts = Proxmox::Install::Config::get_btrfs_opt();
>
>  	    my $mountopts = 'defaults';
> -	    $mountopts .= ",compress=$btrfs_opts->{compress}"
> -		if $btrfs_opts->{compress} ne 'off';
> +	    if ($btrfs_opts->{compress} eq 'on') {
> +		$mountopts .= ',compress';
> +	    } elsif ($btrfs_opts->{compress} ne 'off') {
> +		$mountopts .= ",compress=$btrfs_opts->{compress}";
> +	    }

That was supposed to be squashed into the previous patch, whoops!
I'll send a v2 soon if nothing else comes up, sorry for the noise.