public inbox for pbs-devel@lists.proxmox.com
 help / color / mirror / Atom feed
From: Christian Ebner <c.ebner@proxmox.com>
To: pbs-devel@lists.proxmox.com
Subject: [PATCH v1 00/17] partially fix #6563: add s3 request and traffic counter statistics
Date: Mon,  9 Feb 2026 10:15:16 +0100	[thread overview]
Message-ID: <20260209091533.156902-1-c.ebner@proxmox.com> (raw)

This patch series implements request and traffic counters for the s3
client, being shared as atomic counters via shared memory and mmapping
across all s3-client instances.

The shared counters are instantiated on demand for individual
datastores with s3 backend on s3 client instantiation and stored as
part of the datastore implementation by loading the mmapped file on
datastore instantiation, being cached for further access.

Further, counters are being loaded during rrd metrics collection and
exposed as charts in the datastore summary, including the total
request counts per method, the total upload and download traffic as
well as the averaged upload and download rate.

Usage statistics for the s3 backend are not included in this series
an will be tackled as followup series.

Link to the bugtracker issue:
https://bugzilla.proxmox.com/show_bug.cgi?id=6563


proxmox:

Christian Ebner (6):
  shared-memory: drop check for mmap file being located on tmpfs
  s3-client: add persistent shared request counters for client
  s3-client: add counters for upload/download traffic
  s3-client: account for upload traffic on successful request sending
  s3-client: account for downloaded bytes in incoming response body
  pbs-api-types: define api type for s3 request statistics

 pbs-api-types/src/datastore.rs                |  28 +++
 proxmox-s3-client/Cargo.toml                  |   4 +
 proxmox-s3-client/debian/control              |   2 +
 proxmox-s3-client/examples/s3_client.rs       |   1 +
 proxmox-s3-client/src/client.rs               |  70 +++++-
 proxmox-s3-client/src/lib.rs                  |   7 +-
 proxmox-s3-client/src/response_reader.rs      |  75 +++++-
 .../src/shared_request_counters.rs            | 216 ++++++++++++++++++
 proxmox-shared-memory/src/lib.rs              |   7 -
 9 files changed, 387 insertions(+), 23 deletions(-)
 create mode 100644 proxmox-s3-client/src/shared_request_counters.rs


proxmox-backup:

Christian Ebner (11):
  datastore: collect request statistics for s3 backed datastores
  datastore: expose request counters for s3 backed datastores
  api: s3: add endpoint to reset s3 request counters
  bin: s3: expose request counter reset method as cli command
  datastore: add helper method to get datastore backend type
  ui: improve variable name indirectly fixing typo
  ui: datastore summary: move store to be part of summary panel
  ui: expose s3 request counter statistics in the datastore summary
  metrics: collect s3 datastore statistics as rrd metrics
  api: admin: expose s3 statistics in datastore rrd data
  partially fix #6563: ui: expose s3 rrd charts in datastore summary

 pbs-datastore/src/datastore.rs                |  66 +++-
 pbs-datastore/src/lib.rs                      |   2 +-
 src/api2/admin/datastore.rs                   |  23 +-
 src/api2/admin/s3.rs                          |  86 ++++-
 src/api2/config/s3.rs                         |  18 +-
 src/bin/proxmox_backup_manager/s3.rs          |  33 ++
 src/server/metric_collection/metric_server.rs |   8 +-
 src/server/metric_collection/mod.rs           |  65 +++-
 src/server/metric_collection/pull_metrics.rs  |  18 +-
 src/server/metric_collection/rrd.rs           |  34 +-
 www/datastore/Summary.js                      | 341 ++++++++++++------
 11 files changed, 556 insertions(+), 138 deletions(-)


Summary over all repositories:
  20 files changed, 943 insertions(+), 161 deletions(-)

-- 
Generated by murpp 0.9.0




             reply	other threads:[~2026-02-09  9:15 UTC|newest]

Thread overview: 26+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-02-09  9:15 Christian Ebner [this message]
2026-02-09  9:15 ` [PATCH v1 1/6] shared-memory: drop check for mmap file being located on tmpfs Christian Ebner
2026-02-09  9:15 ` [PATCH v1 2/6] s3-client: add persistent shared request counters for client Christian Ebner
2026-02-11 12:13   ` Robert Obkircher
2026-02-11 12:41     ` Christian Ebner
2026-02-12  9:55       ` Robert Obkircher
2026-02-12 10:19         ` Christian Ebner
2026-02-13 13:37         ` Christian Ebner
2026-02-09  9:15 ` [PATCH v1 3/6] s3-client: add counters for upload/download traffic Christian Ebner
2026-02-09  9:15 ` [PATCH v1 4/6] s3-client: account for upload traffic on successful request sending Christian Ebner
2026-02-09  9:15 ` [PATCH v1 5/6] s3-client: account for downloaded bytes in incoming response body Christian Ebner
2026-02-09  9:15 ` [PATCH v1 6/6] pbs-api-types: define api type for s3 request statistics Christian Ebner
2026-02-09  9:15 ` [PATCH v1 01/11] datastore: collect request statistics for s3 backed datastores Christian Ebner
2026-02-09  9:15 ` [PATCH v1 02/11] datastore: expose request counters " Christian Ebner
2026-02-09  9:15 ` [PATCH v1 03/11] api: s3: add endpoint to reset s3 request counters Christian Ebner
2026-02-09  9:15 ` [PATCH v1 04/11] bin: s3: expose request counter reset method as cli command Christian Ebner
2026-02-09  9:15 ` [PATCH v1 05/11] datastore: add helper method to get datastore backend type Christian Ebner
2026-02-09  9:15 ` [PATCH v1 06/11] ui: improve variable name indirectly fixing typo Christian Ebner
2026-02-09  9:15 ` [PATCH v1 07/11] ui: datastore summary: move store to be part of summary panel Christian Ebner
2026-02-09  9:15 ` [PATCH v1 08/11] ui: expose s3 request counter statistics in the datastore summary Christian Ebner
2026-02-09  9:15 ` [PATCH v1 09/11] metrics: collect s3 datastore statistics as rrd metrics Christian Ebner
2026-02-11 16:29   ` Christian Ebner
2026-02-09  9:15 ` [PATCH v1 10/11] api: admin: expose s3 statistics in datastore rrd data Christian Ebner
2026-02-09  9:15 ` [PATCH v1 11/11] partially fix #6563: ui: expose s3 rrd charts in datastore summary Christian Ebner
2026-02-09  9:39 ` [PATCH v1 00/17] partially fix #6563: add s3 request and traffic counter statistics Christian Ebner
2026-02-16 12:15 ` superseded: " Christian Ebner

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=20260209091533.156902-1-c.ebner@proxmox.com \
    --to=c.ebner@proxmox.com \
    --cc=pbs-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