[pve-devel] [PATCH pve-manager] ui: storage: add is_mountpoint checkmark to directory storage edit

Hannes Laimer h.laimer at proxmox.com
Fri Feb 23 12:59:32 CET 2024


On Fri Feb 23, 2024 at 12:51 PM CET, Aaron Lauterer wrote:
> Overall good idea :)
>
> On 2/23/24 12:45, Hannes Laimer wrote:
> > Signed-off-by: Hannes Laimer <h.laimer at proxmox.com>
> > ---
> > 
> > came up in enterprise support, and I don't think there is a reason to
> > not have it in the UI, while having it in the API
> > 
> >   www/manager6/storage/DirEdit.js | 12 ++++++++++++
> >   1 file changed, 12 insertions(+)
> > 
> > diff --git a/www/manager6/storage/DirEdit.js b/www/manager6/storage/DirEdit.js
> > index 3e2025fc..52e7134b 100644
> > --- a/www/manager6/storage/DirEdit.js
> > +++ b/www/manager6/storage/DirEdit.js
> > @@ -37,6 +37,18 @@ Ext.define('PVE.storage.DirInputPanel', {
> >   	    },
> >   	];
> >   
> > +	me.advancedColumn2 = [
> > +	    {
> > +		xtype: 'proxmoxcheckbox',
> > +		name: 'is_mountpoint',
> > +		uncheckedValue: 0,
> > +		fieldLabel: gettext('Mountpoint'),
> > +		autoEl: {
> > +		    tag: 'div',
> > +		    'data-qtip': gettext('Enable if something is mounted to this directory externally, storage is considered offline as long as nothing is mounted.'),
>
> Maybe phrase the tooltip like this?
>
> Enable if something is mounted at this path. Storage is considered offline as long as nothing is mounted.
sure, I'll send a v2
>
>
> As there are also use-cases where something local is mounted outside of the PVE tooling where the is_mountpoint option is useful.
>
> > +		},
> > +	    },
> > +	];
> >   	me.callParent();
> >       },
> >   });





More information about the pve-devel mailing list