From: Stefan Hanreich <s.hanreich@proxmox.com>
To: pdm-devel@lists.proxmox.com
Subject: [pdm-devel] [PATCH pve-network v3 1/6] sdn: api: return null for rollback / lock endpoints
Date: Tue, 2 Sep 2025 16:09:28 +0200 [thread overview]
Message-ID: <20250902140956.228031-5-s.hanreich@proxmox.com> (raw)
In-Reply-To: <20250902140956.228031-1-s.hanreich@proxmox.com>
lock_sdn_config can return a boolean value, which will then in turn
get returned as data from the API calls. Since we hint type null here,
this leads to problems with the pve-api-client in rust. Fix the return
value for this API call by adding an explicit return statement.
Signed-off-by: Stefan Hanreich <s.hanreich@proxmox.com>
---
src/PVE/API2/Network/SDN.pm | 4 ++++
1 file changed, 4 insertions(+)
diff --git a/src/PVE/API2/Network/SDN.pm b/src/PVE/API2/Network/SDN.pm
index af00b1a..88b229c 100644
--- a/src/PVE/API2/Network/SDN.pm
+++ b/src/PVE/API2/Network/SDN.pm
@@ -192,6 +192,8 @@ __PACKAGE__->register_method({
$param->{'lock-token'},
);
}
+
+ return;
},
});
@@ -247,6 +249,8 @@ __PACKAGE__->register_method({
PVE::Network::SDN::lock_sdn_config(
$rollback, "could not rollback SDN configuration", $lock_token,
);
+
+ return;
},
});
--
2.47.2
_______________________________________________
pdm-devel mailing list
pdm-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pdm-devel
next prev parent reply other threads:[~2025-09-02 14:10 UTC|newest]
Thread overview: 41+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-09-02 14:09 [pdm-devel] [PATCH network/proxmox{, -backup, -api-types, -datacenter-manager} v3 00/30] Add initial SDN / EVPN integration Stefan Hanreich
2025-09-02 14:09 ` [pdm-devel] [PATCH proxmox v3 1/2] schema: use i64 for minimum / maximum / default integer values Stefan Hanreich
2025-09-02 14:09 ` [pdm-devel] [PATCH proxmox v3 2/2] pbs-api-types: fix values for integer schemas Stefan Hanreich
2025-09-02 14:09 ` [pdm-devel] [PATCH proxmox-backup v3 1/1] api: change integer schema parameters to i64 Stefan Hanreich
2025-09-02 14:09 ` Stefan Hanreich [this message]
2025-09-02 14:09 ` [pdm-devel] [PATCH pve-network v3 2/6] controllers: fix maximum value for ASN Stefan Hanreich
2025-09-02 14:09 ` [pdm-devel] [PATCH pve-network v3 3/6] api: add state standard option Stefan Hanreich
2025-09-02 14:09 ` [pdm-devel] [PATCH pve-network v3 4/6] api: controllers: update schema of endpoints Stefan Hanreich
2025-09-02 14:09 ` [pdm-devel] [PATCH pve-network v3 5/6] api: vnets: " Stefan Hanreich
2025-09-02 14:09 ` [pdm-devel] [PATCH pve-network v3 6/6] api: zones: " Stefan Hanreich
2025-09-03 9:37 ` Shannon Sterz
2025-09-03 13:04 ` Stefan Hanreich
2025-09-02 14:09 ` [pdm-devel] [PATCH proxmox-api-types v3 1/6] sdn: add list/create zone endpoints Stefan Hanreich
2025-09-03 9:36 ` Shannon Sterz
2025-09-03 13:15 ` Wolfgang Bumiller
2025-09-02 14:09 ` [pdm-devel] [PATCH proxmox-api-types v3 2/6] sdn: add list/create vnet endpoints Stefan Hanreich
2025-09-03 9:36 ` Shannon Sterz
2025-09-03 12:07 ` Stefan Hanreich
2025-09-03 13:16 ` Wolfgang Bumiller
2025-09-02 14:09 ` [pdm-devel] [PATCH proxmox-api-types v3 3/6] sdn: add list/create controller endpoints Stefan Hanreich
2025-09-02 14:09 ` [pdm-devel] [PATCH proxmox-api-types v3 4/6] sdn: add sdn configuration locking endpoints Stefan Hanreich
2025-09-02 14:09 ` [pdm-devel] [PATCH proxmox-api-types v3 5/6] tasks: add helper for querying successfully finished tasks Stefan Hanreich
2025-09-02 14:09 ` [pdm-devel] [PATCH proxmox-api-types v3 6/6] sdn: add helpers for pending values Stefan Hanreich
2025-09-02 14:09 ` [pdm-devel] [PATCH proxmox-datacenter-manager v3 01/15] server: add locked sdn client helpers Stefan Hanreich
2025-09-03 9:37 ` Shannon Sterz
2025-09-02 14:09 ` [pdm-devel] [PATCH proxmox-datacenter-manager v3 02/15] ui: pve: sdn: add descriptions for sdn tasks Stefan Hanreich
2025-09-02 14:09 ` [pdm-devel] [PATCH proxmox-datacenter-manager v3 03/15] api: sdn: add list_zones endpoint Stefan Hanreich
2025-09-02 14:09 ` [pdm-devel] [PATCH proxmox-datacenter-manager v3 04/15] api: sdn: add create_zone endpoint Stefan Hanreich
2025-09-02 14:09 ` [pdm-devel] [PATCH proxmox-datacenter-manager v3 05/15] api: sdn: add list_vnets endpoint Stefan Hanreich
2025-09-02 14:09 ` [pdm-devel] [PATCH proxmox-datacenter-manager v3 06/15] api: sdn: add create_vnet endpoint Stefan Hanreich
2025-09-02 14:09 ` [pdm-devel] [PATCH proxmox-datacenter-manager v3 07/15] api: sdn: add list_controllers endpoint Stefan Hanreich
2025-09-02 14:09 ` [pdm-devel] [PATCH proxmox-datacenter-manager v3 08/15] ui: sdn: add EvpnRouteTarget type Stefan Hanreich
2025-09-02 14:09 ` [pdm-devel] [PATCH proxmox-datacenter-manager v3 09/15] ui: sdn: add vnet icon Stefan Hanreich
2025-09-02 14:09 ` [pdm-devel] [PATCH proxmox-datacenter-manager v3 10/15] ui: sdn: add view for showing evpn zones Stefan Hanreich
2025-09-02 14:09 ` [pdm-devel] [PATCH proxmox-datacenter-manager v3 11/15] ui: sdn: add view for showing ip vrfs Stefan Hanreich
2025-09-02 14:09 ` [pdm-devel] [PATCH proxmox-datacenter-manager v3 12/15] ui: sdn: add component for creating evpn vnets Stefan Hanreich
2025-09-02 14:09 ` [pdm-devel] [PATCH proxmox-datacenter-manager v3 13/15] ui: sdn: add component for creatin evpn zones Stefan Hanreich
2025-09-02 14:09 ` [pdm-devel] [PATCH proxmox-datacenter-manager v3 14/15] ui: sdn: add evpn overview panel Stefan Hanreich
2025-09-02 14:09 ` [pdm-devel] [PATCH proxmox-datacenter-manager v3 15/15] ui: sdn: add evpn panel to main menu Stefan Hanreich
2025-09-02 14:13 ` [pdm-devel] [PATCH network/proxmox{, -backup, -api-types, -datacenter-manager} v3 00/30] Add initial SDN / EVPN integration Stefan Hanreich
2025-09-03 16:38 ` [pdm-devel] superseded: " Stefan Hanreich
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=20250902140956.228031-5-s.hanreich@proxmox.com \
--to=s.hanreich@proxmox.com \
--cc=pdm-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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.