Re: [pve-devel] [PATCH v2 pve-network] sdn: require mac in IPAM API calls

2023-11-23 Thread Stefan Lendl
sorry, I sent this twice. ___ pve-devel mailing list pve-devel@lists.proxmox.com https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

[pve-devel] [PATCH v2 pve-network] sdn: require mac in IPAM API calls

2023-11-23 Thread Stefan Lendl
standard_option for 'mac-addr' is declared in EVPN where mac-addr is optional. The DELETE endpoint did actually delete from IPAM without a MAC but it would remain in macs.db which brings ends up in a bad state. Signed-off-by: Stefan Lendl --- Changes v2: * removed the if($mac) because we require

[pve-devel] [PATCH v2 pve-network] sdn: require mac in IPAM API calls

2023-11-23 Thread Stefan Lendl
standard_option for 'mac-addr' is declared in EVPN where mac-addr is optional. The DELETE endpoint did actually delete from IPAM without a MAC but it would remain in macs.db which brings ends up in a bad state. Signed-off-by: Stefan Lendl --- Changes v2: * removed the if($mac) because we require