From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from firstgate.proxmox.com (firstgate.proxmox.com [212.224.123.68]) by lore.proxmox.com (Postfix) with ESMTPS id F2A4D1FF139 for ; Tue, 24 Feb 2026 12:51:28 +0100 (CET) Received: from firstgate.proxmox.com (localhost [127.0.0.1]) by firstgate.proxmox.com (Proxmox) with ESMTP id B4FDB7377; Tue, 24 Feb 2026 12:52:24 +0100 (CET) Mime-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset=UTF-8 Date: Tue, 24 Feb 2026 12:51:50 +0100 Message-Id: Subject: Re: [PATCH datacenter-manager/proxmox{,-backup}/widget-toolkit v5 0/9] unstable flag and pdm api viewer To: "Shannon Sterz" X-Mailer: aerc 0.20.0 References: <20251201102852.97396-1-s.sterz@proxmox.com> In-Reply-To: <20251201102852.97396-1-s.sterz@proxmox.com> From: "Shannon Sterz" X-Bm-Milter-Handled: 55990f41-d878-4baa-be0a-ee34c49e34d2 X-Bm-Transport-Timestamp: 1771933894910 X-SPAM-LEVEL: Spam detection results: 0 AWL -2.410 Adjusted score from AWL reputation of From: address BAYES_00 -1.9 Bayes spam probability is 0 to 1% DMARC_MISSING 0.1 Missing DMARC policy 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 URIBL_DBL_SPAM 5 Contains a spam URL listed in the Spamhaus DBL blocklist [macros.rs] Message-ID-Hash: SEPUTHVLMVJR4FS3SDXGXHE5UN7UXJIE X-Message-ID-Hash: SEPUTHVLMVJR4FS3SDXGXHE5UN7UXJIE X-MailFrom: s.sterz@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: pdm-devel@lists.proxmox.com 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: Superseded-by: https://lore.proxmox.com/pbs-devel/20260224115057.140836-1-s= .sterz@proxmox.com/T/#u On Mon Dec 1, 2025 at 11:28 AM CET, Shannon Sterz wrote: > this series adds a flag that marks certain api endpoints as unstable. > allowing for more fine grained control over when and what we declare to > be stable. it also adds an api viewer to pdm as a separate package that > is only suggested for now. > > the code that dumps api definitions as json is also moved to proxmox-rs > and unified between pdm and pbs to reduce technical debt. > > pve-api-types will need to be bumped after wolfgang's commit for it this > to work with pdm. > > Changelog > --------- > > changes since v4: > > * rebase on current master for each repo > * fixed a commit message that wrongly referenced a `stable` instead of > an `unstable` flag (thanks @ Fabian Gr=C3=BCnbichler) > > changes since v3: > > * rebase on current master for each repo. > * drop a patch that was intended for a different series. > > changes since v2 (thanks @ Lukas Wagner): > > * rebase on current master for each repo > * add a doc comment to the public function in proxmox-docgen > > changes since v1: > > * add a patch that fixes a generation bug in pve by Wolfgang Bumiller. > this replaces a previous patch that worked around that bug in > proxmox-docgen. > * add a patch cleaning up some clippy lints in pve-api-types > * clean up some unused import in the docgen binary in pbs & pdm > > proxmox: > > Shannon Sterz (4): > router/api-macro: add unstable flag for ApiMethod > pve-api-types: fix clippy lints > docgen: add docgen crate > docgen: add support for the new unstable flag > > Wolfgang Bumiller (1): > pve-api-types: generate array objects > > Cargo.toml | 1 + > proxmox-api-macro/src/api/method.rs | 9 +- > proxmox-docgen/Cargo.toml | 20 ++ > proxmox-docgen/debian/changelog | 5 + > proxmox-docgen/debian/control | 36 +++ > proxmox-docgen/debian/copyright | 18 ++ > proxmox-docgen/debian/debcargo.toml | 7 + > proxmox-docgen/src/lib.rs | 330 ++++++++++++++++++++++++++++ > proxmox-router/src/router.rs | 10 + > pve-api-types/src/types/macros.rs | 147 ++++++++++++- > pve-api-types/src/types/mod.rs | 3 + > 11 files changed, 583 insertions(+), 3 deletions(-) > create mode 100644 proxmox-docgen/Cargo.toml > create mode 100644 proxmox-docgen/debian/changelog > create mode 100644 proxmox-docgen/debian/control > create mode 100644 proxmox-docgen/debian/copyright > create mode 100644 proxmox-docgen/debian/debcargo.toml > create mode 100644 proxmox-docgen/src/lib.rs > > > proxmox-widget-toolkit: > > Shannon Sterz (1): > api viewer: add support for endpoints that are marked as unstable > > src/api-viewer/APIViewer.js | 18 ++++++++++++++++-- > 1 file changed, 16 insertions(+), 2 deletions(-) > > > proxmox-backup: > > Shannon Sterz (1): > docgen: use proxmox-rs docgen crate > > Cargo.toml | 3 + > docs/api-viewer/index.html | 2 + > src/bin/docgen.rs | 326 ++----------------------------------- > 3 files changed, 20 insertions(+), 311 deletions(-) > > > proxmox-datacenter-manager: > > Shannon Sterz (2): > docgen: switch to proxmox-rs docgen crate > api-viewer: add an api-viewer package > > Cargo.toml | 2 + > Makefile | 1 + > debian/control | 13 + > ...xmox-datacenter-manager-api-viewer.install | 1 + > docs/api-viewer/Makefile | 27 ++ > docs/api-viewer/index.html | 16 + > server/Cargo.toml | 1 + > server/src/bin/docgen.rs | 314 +----------------- > 8 files changed, 65 insertions(+), 310 deletions(-) > create mode 100644 debian/proxmox-datacenter-manager-api-viewer.install > create mode 100644 docs/api-viewer/Makefile > create mode 100644 docs/api-viewer/index.html > > > Summary over all repositories: > 23 files changed, 684 insertions(+), 626 deletions(-) > > -- > Generated by git-murpp 0.8.1