From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from gate001.proxmox.com (gate001.proxmox.com [IPv6:2a0f:8001:1:32::40]) by lore.proxmox.com (Postfix) with ESMTPS id DE8F41FF0E2 for ; Thu, 30 Jul 2026 12:13:59 +0200 (CEST) Received: from gate001.proxmox.com (localhost.localdomain [127.0.0.1]) by gate001.proxmox.com (Proxmox) with ESMTP id D0E3D214B4; Thu, 30 Jul 2026 12:13:55 +0200 (CEST) From: Erik Fastermann To: pbs-devel@lists.proxmox.com, pdm-devel@lists.proxmox.com Subject: [PATCH proxmox{,-backup,-datacenter-manager} v2 0/5] factor system report into shared crate Date: Thu, 30 Jul 2026 12:13:44 +0200 Message-ID: <20260730101349.217371-1-e.fastermann@proxmox.com> X-Mailer: git-send-email 2.47.3 MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit X-SPAM-LEVEL: Spam detection results: 1 AWL -0.487 Adjusted score from AWL reputation of From: address DMARC_MISSING 0.1 Missing DMARC policy KAM_DMARC_STATUS 0.01 Test Rule for DKIM or SPF Failure with Strict Alignment (newer systems) KAM_LAZY_DOMAIN_SECURITY 1 Sending domain does not have any anti-forgery methods RDNS_NONE 1.274 Delivered to internal network by a host with no rDNS SPF_HELO_NONE 0.001 SPF: HELO does not publish an SPF Record SPF_NONE 0.001 SPF: sender does not publish an SPF Record Message-ID-Hash: W4VELGIVVI6D34G2UWIZQO5BADGCOM33 X-Message-ID-Hash: W4VELGIVVI6D34G2UWIZQO5BADGCOM33 X-MailFrom: efastermann@ruth.proxmox.com X-Mailman-Rule-Misses: dmarc-mitigation; no-senders; approved; loop; banned-address; emergency; member-moderation; nonmember-moderation; administrivia; implicit-dest; max-recipients; max-size; news-moderation; no-subject; digests; suspicious-header CC: Erik Fastermann X-Mailman-Version: 3.3.10 Precedence: list List-Id: Proxmox Datacenter Manager development discussion List-Help: List-Owner: List-Post: List-Subscribe: List-Unsubscribe: This series moves the system report logic that was duplicated in PBS and PDM into a new shared proxmox-system-report crate. proxmox: Adds the proxmox-system-report crate holding the common file, command and function collection, the unified section order (FILES, COMMANDS, FUNCTIONS) and the merged set of general commands run on every product. proxmox-backup, proxmox-datacenter-manager: Drop the duplicated logic and generate the report through the shared crate, passing only their product specific files, commands and functions. The report content stays the same apart from two intentional effects of sharing one implementation: the top-level section order is now the same for both products, and the general command set is the union of what each ran before. PDM additionally emits FILES first now instead of last. Product specific entries are untouched. Changes since v1: * Use version 1.0.0 for the new crate * Add Debian changelog and control files for the new crate * Use only 2026 as the year for the copyright notice of the new crate * Rename CommandSpec types and update the doc comment * Use the common_* and project_* prefixes for the related functions * Add the previously missing report dependencies as extra commits for PDM and PBS * Add the newly required report dependencies to PDM * Add librust-proxmox-system-report-1+default-dev as a dependency to PDM and PBS * Add a commented out local path override for proxmox-system-report to the `Cargo.toml` file in PDM and PBS Thank you @Christian Ebner for the feedback on the first version. proxmox: Erik Fastermann (1): system-report: add crate for shared report generation Cargo.toml | 2 + proxmox-system-report/Cargo.toml | 16 ++ proxmox-system-report/debian/changelog | 6 + proxmox-system-report/debian/control | 36 ++++ proxmox-system-report/debian/copyright | 18 ++ proxmox-system-report/debian/debcargo.toml | 7 + proxmox-system-report/src/lib.rs | 239 +++++++++++++++++++++ 7 files changed, 324 insertions(+) create mode 100644 proxmox-system-report/Cargo.toml create mode 100644 proxmox-system-report/debian/changelog create mode 100644 proxmox-system-report/debian/control create mode 100644 proxmox-system-report/debian/copyright create mode 100644 proxmox-system-report/debian/debcargo.toml create mode 100644 proxmox-system-report/src/lib.rs proxmox-backup: Erik Fastermann (2): d/control: depend on tools used by the system report report: use shared proxmox-system-report crate Cargo.toml | 3 + debian/control | 4 + src/server/report.rs | 217 ++++--------------------------------------- 3 files changed, 24 insertions(+), 200 deletions(-) proxmox-datacenter-manager: Erik Fastermann (2): d/control: depend on tools used by the system report report: use shared proxmox-system-report crate Cargo.toml | 2 + debian/control | 10 ++- server/Cargo.toml | 1 + server/src/report.rs | 198 +------------------------------------------ 4 files changed, 16 insertions(+), 195 deletions(-) Summary over all repositories: 14 files changed, 364 insertions(+), 395 deletions(-) -- Generated by murpp 0.11.0