public inbox for pve-devel@lists.proxmox.com
 help / color / mirror / Atom feed
* [pve-devel] [PATCH] api: sdn: ipams: status: ensure that guest ID is returned as an integer
@ 2025-08-01 11:44 Fiona Ebner
  2025-08-01 11:47 ` Fiona Ebner
  2025-08-01 13:18 ` Thomas Lamprecht
  0 siblings, 2 replies; 3+ messages in thread
From: Fiona Ebner @ 2025-08-01 11:44 UTC (permalink / raw)
  To: pve-devel

This is a breaking API change intended for PVE 9.

Signed-off-by: Fiona Ebner <f.ebner@proxmox.com>
---

Together with [0] this makes sorting numerically by guest ID work in
the SDN IPAM UI.

[0] https://lore.proxmox.com/pve-devel/2014f691-aa28-4ab4-91c1-48c1f68d8a54@proxmox.com/T/

 src/PVE/API2/Network/SDN/Ipams.pm | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/src/PVE/API2/Network/SDN/Ipams.pm b/src/PVE/API2/Network/SDN/Ipams.pm
index 89e05ee..9955817 100644
--- a/src/PVE/API2/Network/SDN/Ipams.pm
+++ b/src/PVE/API2/Network/SDN/Ipams.pm
@@ -364,6 +364,8 @@ __PACKAGE__->register_method({
                     $entry->{ip} = $ip;
                     $entry->{vnet} = $vnet;
 
+                    $entry->{vmid} = int($entry->{vmid}) if defined($entry->{vmid});
+
                     push @$result, $entry;
                 }
             }
-- 
2.47.2



_______________________________________________
pve-devel mailing list
pve-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [pve-devel] [PATCH] api: sdn: ipams: status: ensure that guest ID is returned as an integer
  2025-08-01 11:44 [pve-devel] [PATCH] api: sdn: ipams: status: ensure that guest ID is returned as an integer Fiona Ebner
@ 2025-08-01 11:47 ` Fiona Ebner
  2025-08-01 13:18 ` Thomas Lamprecht
  1 sibling, 0 replies; 3+ messages in thread
From: Fiona Ebner @ 2025-08-01 11:47 UTC (permalink / raw)
  To: pve-devel

This is for the pve-network repository. Sorry, seems like I haven't
configured the prefix for this in my git config yet.

Am 01.08.25 um 1:45 PM schrieb Fiona Ebner:
> This is a breaking API change intended for PVE 9.
> 
> Signed-off-by: Fiona Ebner <f.ebner@proxmox.com>
> ---
> 
> Together with [0] this makes sorting numerically by guest ID work in
> the SDN IPAM UI.
> 
> [0] https://lore.proxmox.com/pve-devel/2014f691-aa28-4ab4-91c1-48c1f68d8a54@proxmox.com/T/
> 
>  src/PVE/API2/Network/SDN/Ipams.pm | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/src/PVE/API2/Network/SDN/Ipams.pm b/src/PVE/API2/Network/SDN/Ipams.pm
> index 89e05ee..9955817 100644
> --- a/src/PVE/API2/Network/SDN/Ipams.pm
> +++ b/src/PVE/API2/Network/SDN/Ipams.pm
> @@ -364,6 +364,8 @@ __PACKAGE__->register_method({
>                      $entry->{ip} = $ip;
>                      $entry->{vnet} = $vnet;
>  
> +                    $entry->{vmid} = int($entry->{vmid}) if defined($entry->{vmid});
> +
>                      push @$result, $entry;
>                  }
>              }



_______________________________________________
pve-devel mailing list
pve-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [pve-devel] [PATCH] api: sdn: ipams: status: ensure that guest ID is returned as an integer
  2025-08-01 11:44 [pve-devel] [PATCH] api: sdn: ipams: status: ensure that guest ID is returned as an integer Fiona Ebner
  2025-08-01 11:47 ` Fiona Ebner
@ 2025-08-01 13:18 ` Thomas Lamprecht
  1 sibling, 0 replies; 3+ messages in thread
From: Thomas Lamprecht @ 2025-08-01 13:18 UTC (permalink / raw)
  To: pve-devel, Fiona Ebner

On Fri, 01 Aug 2025 13:44:45 +0200, Fiona Ebner wrote:
> This is a breaking API change intended for PVE 9.
> 
> 

Applied, thanks!

[1/1] api: sdn: ipams: status: ensure that guest ID is returned as an integer
      commit: 37c2156ce8b26d6fb5e6e173b660841eb64a83cd


_______________________________________________
pve-devel mailing list
pve-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel


^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2025-08-01 13:33 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2025-08-01 11:44 [pve-devel] [PATCH] api: sdn: ipams: status: ensure that guest ID is returned as an integer Fiona Ebner
2025-08-01 11:47 ` Fiona Ebner
2025-08-01 13:18 ` Thomas Lamprecht

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
Service provided by Proxmox Server Solutions GmbH | Privacy | Legal