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

Fiona Ebner f.ebner at proxmox.com
Mon Apr 15 15:10:31 CEST 2024


Am 23.02.24 um 13:03 schrieb Hannes Laimer:
> Signed-off-by: Hannes Laimer <h.laimer at proxmox.com>

Reviewed-by: Fiona Ebner <f.ebner 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
> 

This rationale could/should become the commit message, i.e. the fact
that having it in the UI is convenient for many users.

The plugin for BTRFS also supports this option. I think it makes sense
adding it there too. Could even be done in storage/Base.js like for
preallocation, but as long as there is no third plugin with the option,
duplication is also fine IMHO.




More information about the pve-devel mailing list