From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from firstgate.proxmox.com (firstgate.proxmox.com [212.224.123.68]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits)) (No client certificate requested) by lists.proxmox.com (Postfix) with ESMTPS id F0C1284B0A for ; Wed, 15 Dec 2021 10:20:33 +0100 (CET) Received: from firstgate.proxmox.com (localhost [127.0.0.1]) by firstgate.proxmox.com (Proxmox) with ESMTP id E86F1FB87 for ; Wed, 15 Dec 2021 10:20:03 +0100 (CET) Received: from proxmox-new.maurer-it.com (proxmox-new.maurer-it.com [94.136.29.106]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits)) (No client certificate requested) by firstgate.proxmox.com (Proxmox) with ESMTPS id 88A9EFAAF for ; Wed, 15 Dec 2021 10:20:00 +0100 (CET) Received: from proxmox-new.maurer-it.com (localhost.localdomain [127.0.0.1]) by proxmox-new.maurer-it.com (Proxmox) with ESMTP id 25E1D45201 for ; Wed, 15 Dec 2021 10:19:54 +0100 (CET) From: Dominik Csapak To: pbs-devel@lists.proxmox.com Date: Wed, 15 Dec 2021 10:19:43 +0100 Message-Id: <20211215091952.1490583-1-d.csapak@proxmox.com> X-Mailer: git-send-email 2.30.2 MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-SPAM-LEVEL: Spam detection results: 0 AWL 0.175 Adjusted score from AWL reputation of From: address BAYES_00 -1.9 Bayes spam probability is 0 to 1% KAM_DMARC_STATUS 0.01 Test Rule for DKIM or SPF Failure with Strict Alignment SPF_HELO_NONE 0.001 SPF: HELO does not publish an SPF Record SPF_PASS -0.001 SPF: sender matches SPF record Subject: [pbs-devel] [PATCH proxmox/proxmox-backup v2] add metrics server capability X-BeenThere: pbs-devel@lists.proxmox.com X-Mailman-Version: 2.1.29 Precedence: list List-Id: Proxmox Backup Server development discussion List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 15 Dec 2021 09:20:34 -0000 this series adds support for exporting metrics data to external metric servers. for now this includes only data we gather for RRD, though it should not be hard to extend that functionality also only influxdb (udp/http(s)) is currently supported, but it should also not be too hard to include more options here i did not include gui/cli patches yet, as i find the proxmox-backup-manager options are already too much and i waited for the gui for some feedback. for testing, the metric servers can be added either by calling 'proxmox-backup debug api ...' or by manually editing the file a bit unrelated: i moved the 'disk_usage' method to proxmox-sys (fits better there, and we want to use that from other things too) ofc, proxmox-backup depends on bumped versions of the proxmox-* crates changes from v1: * fixed ipv6 support for udp (tested it this time ;) ) * dropped the 'flush' functionality of the MetricsChannel, but kept the wrapper struct: it did not do what i intended, and after rethinking it, turns out it's not necessary (as we autoflush when the data gets to large, or when we close the channel). kept the struct so that the interface can stay the same even if we want to implement a manual flush in the future * improved the influxdb line formatter * removed variables like 'names2' by reorganizing the code * used Arc::clone(&foo) instead of foo.clone() (better visibilty) * used CamelCase for the DeletableProperties proxmox: Dominik Csapak (3): proxmox-sys: make some structs serializable proxmox-sys: add DiskUsage struct and helper proxmox-metrics: implement metrics server client code Cargo.toml | 1 + proxmox-metrics/Cargo.toml | 20 ++++ proxmox-metrics/debian/changelog | 5 + proxmox-metrics/debian/copyright | 16 ++++ proxmox-metrics/debian/debcargo.toml | 7 ++ proxmox-metrics/src/influxdb/http.rs | 127 ++++++++++++++++++++++++++ proxmox-metrics/src/influxdb/mod.rs | 7 ++ proxmox-metrics/src/influxdb/udp.rs | 98 ++++++++++++++++++++ proxmox-metrics/src/influxdb/utils.rs | 52 +++++++++++ proxmox-metrics/src/lib.rs | 73 +++++++++++++++ proxmox-sys/Cargo.toml | 1 + proxmox-sys/src/fs/mod.rs | 26 ++++++ proxmox-sys/src/linux/procfs/mod.rs | 7 +- 13 files changed, 437 insertions(+), 3 deletions(-) create mode 100644 proxmox-metrics/Cargo.toml create mode 100644 proxmox-metrics/debian/changelog create mode 100644 proxmox-metrics/debian/copyright create mode 100644 proxmox-metrics/debian/debcargo.toml create mode 100644 proxmox-metrics/src/influxdb/http.rs create mode 100644 proxmox-metrics/src/influxdb/mod.rs create mode 100644 proxmox-metrics/src/influxdb/udp.rs create mode 100644 proxmox-metrics/src/influxdb/utils.rs create mode 100644 proxmox-metrics/src/lib.rs proxmox-backup: Dominik Csapak (6): use 'disk_usage' from proxmox-sys pbs-api-types: add metrics api types pbs-config: add metrics config class backup-proxy: decouple stats gathering from rrd update proxmox-backup-proxy: send metrics to configured metrics server api: add metricserver endpoints Cargo.toml | 1 + pbs-api-types/src/lib.rs | 2 + pbs-api-types/src/metrics.rs | 134 ++++++++ pbs-config/Cargo.toml | 1 + pbs-config/src/lib.rs | 1 + pbs-config/src/metrics.rs | 122 +++++++ src/api2/admin/datastore.rs | 4 +- src/api2/config/metricserver/influxdbhttp.rs | 271 +++++++++++++++ src/api2/config/metricserver/influxdbudp.rs | 241 +++++++++++++ src/api2/config/metricserver/mod.rs | 16 + src/api2/config/mod.rs | 2 + src/api2/node/status.rs | 10 +- src/api2/status.rs | 4 +- src/bin/proxmox-backup-proxy.rs | 344 +++++++++++++++---- src/tools/disks/mod.rs | 21 +- 15 files changed, 1077 insertions(+), 97 deletions(-) create mode 100644 pbs-api-types/src/metrics.rs create mode 100644 pbs-config/src/metrics.rs create mode 100644 src/api2/config/metricserver/influxdbhttp.rs create mode 100644 src/api2/config/metricserver/influxdbudp.rs create mode 100644 src/api2/config/metricserver/mod.rs -- 2.30.2