From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from firstgate.proxmox.com (firstgate.proxmox.com [IPv6:2a01:7e0:0:424::9]) by lore.proxmox.com (Postfix) with ESMTPS id D3B4C1FF17E for ; Thu, 13 Nov 2025 13:02:13 +0100 (CET) Received: from firstgate.proxmox.com (localhost [127.0.0.1]) by firstgate.proxmox.com (Proxmox) with ESMTP id CA3381A169; Thu, 13 Nov 2025 13:03:08 +0100 (CET) Mime-Version: 1.0 Date: Thu, 13 Nov 2025 13:02:35 +0100 To: "Shannon Sterz" Message-Id: X-Mailer: aerc 0.20.0 References: <20251112102415.175358-1-s.sterz@proxmox.com> In-Reply-To: <20251112102415.175358-1-s.sterz@proxmox.com> From: "Shannon Sterz" X-Bm-Milter-Handled: 55990f41-d878-4baa-be0a-ee34c49e34d2 X-Bm-Transport-Timestamp: 1763035329673 X-SPAM-LEVEL: Spam detection results: 0 AWL 0.062 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 Subject: [pdm-devel] Superseded: Re: [PATCH datacenter-manager/proxmox{, -backup}/widget-toolkit 0/8] unstable flag and pdm api viewer X-BeenThere: pdm-devel@lists.proxmox.com X-Mailman-Version: 2.1.29 Precedence: list List-Id: Proxmox Datacenter Manager development discussion List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Reply-To: Proxmox Datacenter Manager development discussion Cc: pdm-devel@lists.proxmox.com Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: pdm-devel-bounces@lists.proxmox.com Sender: "pdm-devel" Superseded-by: https://lore.proxmox.com/pdm-devel/20251113120021.331639-1-s.sterz@proxmox.com/ On Wed Nov 12, 2025 at 11:24 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. > > Follow Ups > ---------- > > * currently the proxmox-docgen handles AllOf schemas that have > additional properties that are not object schema to always have > additoinal properties. otherwise, a panic would be caused in > proxmox-schema. the proper fix here is to adapt schema generation in > pve-api-types that is caused by some properties being generated as > string schemas that wrap a property string. > > the correct way of handling those would be to either fix > proxmox-schema to also handle these types of schema without panicing > or to not generate these type of schema in the first place. > > 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: > > Shannon Sterz (4): > router/api-macro: add unstable flag for ApiMethod > docgen: add docgen crate > docgen: add support for the new stable flag > docgen: add a stop-gap fix to allow generating schema for > pve-api-types > > 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 | 339 ++++++++++++++++++++++++++++ > proxmox-router/src/router.rs | 10 + > 9 files changed, 444 insertions(+), 1 deletion(-) > 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-backup: > > Shannon Sterz (1): > docgen: use proxmox-rs docgen crate > > Cargo.toml | 3 + > docs/api-viewer/index.html | 2 + > src/bin/docgen.rs | 312 +------------------------------------ > 3 files changed, 10 insertions(+), 307 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 | 308 +----------------- > 8 files changed, 63 insertions(+), 306 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: > 21 files changed, 533 insertions(+), 616 deletions(-) > > -- > Generated by git-murpp 0.8.1 _______________________________________________ pdm-devel mailing list pdm-devel@lists.proxmox.com https://lists.proxmox.com/cgi-bin/mailman/listinfo/pdm-devel