public inbox for pdm-devel@lists.proxmox.com
 help / color / mirror / Atom feed
From: Lukas Wagner <l.wagner@proxmox.com>
To: pdm-devel@lists.proxmox.com
Subject: [PATCH datacenter-manager/proxmox-yew-comp v4 00/12] metric collection for the PDM host
Date: Wed, 15 Apr 2026 15:20:01 +0200	[thread overview]
Message-ID: <20260415132013.440581-1-l.wagner@proxmox.com> (raw)

This series add metric collection physical PDM hosts.

The patches for proxmox-yew-comp slight adapt the existing NodeStatusPanel to allow the application
to inject child components into the same panel.

The proxmox-datacenter-manager patches do some initial refactoring (naming), and then add the needed
collection loop, API types and UI elements.

Changes since v3:
  - moved RRD value render helpers to yew-comp (thx Shannon!)
  - included Michael's T-b/R-b trailers! (thx!)

Changes since v2:
  - rebased
  - adapted PDM part to changes in proxmox-disks (minor, only renaming)
  - drop already applied patches for 'proxmox'
  - drop patches for 'proxmox-backup' - these need to be adapted for the refactoring of
    proxmox-disks and are pretty much independent of this series here, so they will be posted
    separately

Changes since v1:
  - rebased
  - fixed failing unit test
  - fixed mistake in Cargo.toml
  - dropped patch for proxmox-sys, Fabian fixed the issue
    using another approach


proxmox-yew-comp:

Lukas Wagner (4):
  node status panel: add `children` property
  RRDGrid: fix size observer by attaching node reference to rendered
    container
  RRDGrid: add padding and increase gap between elements
  rrd: add rrd_value_renderer module

 src/lib.rs                    |  2 +-
 src/node_status_panel.rs      | 16 +++++++
 src/rrd/mod.rs                |  2 +
 src/rrd/rrd_value_renderer.rs | 84 +++++++++++++++++++++++++++++++++++
 src/rrd_grid.rs               |  5 ++-
 5 files changed, 106 insertions(+), 3 deletions(-)
 create mode 100644 src/rrd/rrd_value_renderer.rs


proxmox-datacenter-manager:

Lukas Wagner (8):
  metric collection: clarify naming for remote metric collection
  metric collection: fix minor typo in error message
  metric collection: collect PDM host metrics in a new collection task
  api: fix /nodes/localhost/rrddata endpoint
  pdm: node rrd data: rename 'total-time' to
    'metric-collection-total-time'
  pdm-api-types: add PDM host metric fields
  ui: node status: add RRD graphs for PDM host metrics
  ui: lxc/qemu/node: use RRD value render helpers from yew-comp

 Cargo.toml                                    |   2 +
 cli/client/src/metric_collection.rs           |   4 +-
 debian/control                                |   2 +
 lib/pdm-api-types/src/metric_collection.rs    |   2 +-
 lib/pdm-api-types/src/rrddata.rs              |  74 ++++-
 lib/pdm-client/src/lib.rs                     |   8 +-
 server/Cargo.toml                             |   2 +
 server/src/api/metric_collection.rs           |  10 +-
 server/src/api/nodes/mod.rs                   |   2 +-
 server/src/api/nodes/rrddata.rs               |  73 +++-
 server/src/api/remotes.rs                     |   2 +-
 server/src/api/rrd_common.rs                  |   2 +-
 .../local_collection_task.rs                  | 199 +++++++++++
 server/src/metric_collection/mod.rs           |  40 ++-
 ...tion_task.rs => remote_collection_task.rs} |   8 +-
 server/src/metric_collection/rrd_task.rs      | 187 ++++++++++-
 server/src/metric_collection/state.rs         |   2 +-
 ui/src/administration/node_status.rs          | 312 +++++++++++++++++-
 ui/src/pbs/node/overview.rs                   |  28 +-
 ui/src/pve/lxc/overview.rs                    |  34 +-
 ui/src/pve/node/overview.rs                   |  28 +-
 ui/src/pve/qemu/overview.rs                   |  34 +-
 22 files changed, 904 insertions(+), 151 deletions(-)
 create mode 100644 server/src/metric_collection/local_collection_task.rs
 rename server/src/metric_collection/{collection_task.rs => remote_collection_task.rs} (99%)


Summary over all repositories:
  27 files changed, 1010 insertions(+), 154 deletions(-)

-- 
Generated by murpp 0.12.0




             reply	other threads:[~2026-04-15 13:21 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-04-15 13:20 Lukas Wagner [this message]
2026-04-15 13:20 ` [PATCH proxmox-yew-comp v4 01/12] node status panel: add `children` property Lukas Wagner
2026-04-15 13:20 ` [PATCH proxmox-yew-comp v4 02/12] RRDGrid: fix size observer by attaching node reference to rendered container Lukas Wagner
2026-04-15 13:20 ` [PATCH proxmox-yew-comp v4 03/12] RRDGrid: add padding and increase gap between elements Lukas Wagner
2026-04-15 13:20 ` [PATCH proxmox-yew-comp v4 04/12] rrd: add rrd_value_renderer module Lukas Wagner
2026-04-15 13:20 ` [PATCH datacenter-manager v4 05/12] metric collection: clarify naming for remote metric collection Lukas Wagner
2026-04-15 13:20 ` [PATCH datacenter-manager v4 06/12] metric collection: fix minor typo in error message Lukas Wagner
2026-04-15 13:20 ` [PATCH datacenter-manager v4 07/12] metric collection: collect PDM host metrics in a new collection task Lukas Wagner
2026-04-15 13:20 ` [PATCH datacenter-manager v4 08/12] api: fix /nodes/localhost/rrddata endpoint Lukas Wagner
2026-04-15 13:20 ` [PATCH datacenter-manager v4 09/12] pdm: node rrd data: rename 'total-time' to 'metric-collection-total-time' Lukas Wagner
2026-04-15 13:20 ` [PATCH datacenter-manager v4 10/12] pdm-api-types: add PDM host metric fields Lukas Wagner
2026-04-15 13:20 ` [PATCH datacenter-manager v4 11/12] ui: node status: add RRD graphs for PDM host metrics Lukas Wagner
2026-04-15 13:20 ` [PATCH datacenter-manager v4 12/12] ui: lxc/qemu/node: use RRD value render helpers from yew-comp Lukas Wagner

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=20260415132013.440581-1-l.wagner@proxmox.com \
    --to=l.wagner@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