[pbs-devel] applied-series: [PATCH proxmox-backup/proxmox-widget-toolkit v2 0/4] add partitions to disks/list endpointg

Wolfgang Bumiller w.bumiller at proxmox.com
Wed Jun 15 11:09:51 CEST 2022


applied series, thanks

On Wed, Jun 08, 2022 at 08:51:50AM +0000, Hannes Laimer wrote:
> In order to work with the existing frontend component for displying the
> disklist, either
> * the partition data has to be return with the same struct a disk is
>   returned. Which leads to the same struct being used for different
>   things -> quite a few fields are always empty for partitions and a new
>   'type' field would be needed. Also the code structure in PBS has to be
>   changed quite a bit.
> * the existing frontend has to be adjusted to handle data from PVE and
>   PBS properly.
> 
> I went with the second option because the adjustments nedded in the UI
> compenent were minimal and, IMHO, adjusting the API to fit the UI is the
> wrong direction.
> 
> For the mount status to shown in the UI, the patch[1] sent to pve-devel for
> the 'mounted' column is needed.
> 
> NOTE: The partition data will be needed in later patches for removable
> datastores.
> 
> v1->v2:
>  * use builder pattern for queries as suggested by Wolfgang
>  * move mounted out of Enum and add filesystem string
>  * add missing zfsreserve usage type
>  * add 'mounted' column to disklist (separate patch[1])
> 
> 
> [1] https://lists.proxmox.com/pipermail/pve-devel/2022-June/053211.html
> 
> * proxmox-backup
> Hannes Laimer (3):
>   api2: disks endpoint return partitions
>   disks: use builder pattern for querying disk usage
>   ui: disks: show partitions by default
> 
>  src/api2/node/disks/directory.rs |   6 +-
>  src/api2/node/disks/mod.rs       |  19 ++-
>  src/api2/node/disks/zfs.rs       |   2 +-
>  src/tools/disks/mod.rs           | 206 +++++++++++++++++++++++++++++--
>  www/panel/StorageAndDisks.js     |   1 +
>  5 files changed, 215 insertions(+), 19 deletions(-)
> 
> * proxmox-widget-toolkit
> Hannes Laimer (1):
>   ui: DiskLisk: handle partition data from PBS backend
> 
>  src/panel/DiskList.js | 22 ++++++++++++++++++++--
>  1 file changed, 20 insertions(+), 2 deletions(-)
> 
> -- 
> 2.30.2





More information about the pbs-devel mailing list