public inbox for pdm-devel@lists.proxmox.com
 help / color / mirror / Atom feed
From: Lukas Sichert <l.sichert@proxmox.com>
To: pdm-devel@lists.proxmox.com
Cc: Lukas Sichert <l.sichert@proxmox.com>
Subject: [PATCH api-types/manager/datacenter-manager 0/4] fix #7315: datacenter-manager: fix wrong calculation of total storage
Date: Mon,  2 Mar 2026 14:45:33 +0100	[thread overview]
Message-ID: <20260302134537.108696-1-l.sichert@proxmox.com> (raw)

Currently, the calculation for total storage in a cluster with shared
storage is incorrect. For every node to which the shared storage is
configured, the storage capacity is added to the total capacity.

In order to perform the correct calculations, the pve backend must
provide a "shared" flag to the datacenter-manager backend. The PVE
backend already responds with the flag. However, since the flag is not
described in the JSON schema, the Rust backend cannot extract the value.

Currently, regenerating the Rust files, where the schema is described
for the Rust backend, is not possible because a new Windows certificate
has been added. 

Threfore in this series:
  -the generate script is updated,
  -the generated files after running 'make refresh' are updated,
  -the shared flag is added to the json schema,
  -the storage calculations are corrected using the shared flag.

proxmox:
Lukas Sichert (2):
  fix #7315: generate: add support for new certificate variant
  fix #7315: run make refresh

 pve-api-types/generate.pl            |  1 +
 pve-api-types/pve-api.json           | 85 ++++++++++++++++++++-----
 pve-api-types/src/generated/types.rs | 92 +++++++++++++++++++++++++---
 3 files changed, 154 insertions(+), 24 deletions(-)

pve-manager:
Lukas Sichert (1):
  fix #7315: add missing storage flag to json Schema

 PVE/API2/Cluster.pm | 5 +++++
 1 file changed, 5 insertions(+)


proxmox-datacenter-manager:
Lukas Sichert (1):
  fix #7135: ui: correct calculations for shared storages

 cli/client/src/resources.rs       | 1 +
 lib/pdm-api-types/src/resource.rs | 2 ++
 server/src/api/resources.rs       | 1 +
 server/src/views/tests.rs         | 1 +
 ui/src/pve/remote_overview.rs     | 9 +++++++--
 5 files changed, 12 insertions(+), 2 deletions(-)


-- 
2.47.3




             reply	other threads:[~2026-03-02 13:45 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-03-02 13:45 Lukas Sichert [this message]
2026-03-02 13:45 ` [PATCH proxmox 1/4] fix #7315: generate: add support for new certificate variant Lukas Sichert
2026-03-02 13:45 ` SPAM: [PATCH proxmox 2/4] fix #7315: run make refresh Lukas Sichert
2026-03-02 13:45 ` [PATCH manager 3/4] fix #7315: types: add missing storage flag to json schema Lukas Sichert
2026-03-02 13:51   ` Dominik Rusovac
2026-03-02 13:45 ` [PATCH datacenter-manager 4/4] fix #7135: ui: correct calculations for shared storages Lukas Sichert

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20260302134537.108696-1-l.sichert@proxmox.com \
    --to=l.sichert@proxmox.com \
    --cc=pdm-devel@lists.proxmox.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
Service provided by Proxmox Server Solutions GmbH | Privacy | Legal