Re: [PATCH manager 2/2] ui: lxc/MPEdit: add "keepattrs" flag

["Fabian Grünbichler" <f.gruenbichler@proxmox.com>]

On February 11, 2026 12:06 pm, Filip Schauer wrote:
> Expose the "keepattrs" flag for mount points in the UI.
> 
> Signed-off-by: Filip Schauer <f.schauer@proxmox.com>

Reviewed-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>

> ---
> This depends on patch 1/2.
> 
>  www/manager6/lxc/MPEdit.js | 15 +++++++++++++++
>  1 file changed, 15 insertions(+)
> 
> diff --git a/www/manager6/lxc/MPEdit.js b/www/manager6/lxc/MPEdit.js
> index f4c45876..4bba536c 100644
> --- a/www/manager6/lxc/MPEdit.js
> +++ b/www/manager6/lxc/MPEdit.js
> @@ -47,6 +47,7 @@ Ext.define('PVE.lxc.MountPointInputPanel', {
>          setMPOpt('ro', values.ro);
>          setMPOpt('acl', values.acl);
>          setMPOpt('replicate', values.replicate);
> +        setMPOpt('keepattrs', values.keepattrs);
>  
>          let res = {};
>          res[confid] = PVE.Parser.printLxcMountPoint(me.mp);
> @@ -338,6 +339,20 @@ Ext.define('PVE.lxc.MountPointInputPanel', {
>              name: 'replicate',
>              fieldLabel: gettext('Skip replication'),
>          },
> +        {
> +            xtype: 'proxmoxcheckbox',
> +            name: 'keepattrs',
> +            defaultValue: '0',
> +            fieldLabel: gettext('Keep Attributes'),
> +            autoEl: {
> +                tag: 'div',
> +                'data-qtip': gettext('Preserve Attributes of Target Directory'),

nit: this is not a heading/label, so it should have regular casing?

> +            },
> +            bind: {
> +                hidden: '{isRoot}',
> +                disabled: '{isRoot}',
> +            },
> +        },
>      ],
>  });
>  
> -- 
> 2.47.3
> 
> 
> 
> 
> 
>