--- Begin Message ---
This fixes #6569 [1]. It basically changes the Netbox lookup from a search to a
lookup. In Netbox it is possible to have multiple registrations of the same
prefix as long as they belong to different VRFs, which is a case this patch also
covers.
I'm not entirely sure the current logic by failing if the prefix exists is the
best, it's easy to think of cases where the prefixes would be planned in Netbox
first (either manually or via automation). But I guess that is a discussion for
another day.
This is my first patch to Proxmox, please let me know if I should do things in a
different way. I will sign the CLA in the following days when I have access to a
scanner.
[1]: https://bugzilla.proxmox.com/show_bug.cgi?id=6569
[2]:
https://git.proxmox.com/?p=pve-network.git;a=blob;f=src/PVE/Network/SDN/Ipams/NetboxPlugin.pm;h=e118d03e1eb062d0b69cbb53bb7eb875a1e55336;hb=ea6db24afdaf63dedcdd4d74ce97918f7f82b034#l81
Trygve Laugstøl (1):
fix #6569: ipam: netbox: better prefix lookup
src/PVE/Network/SDN/Ipams/NetboxPlugin.pm | 21 ++++++++++++++-------
1 file changed, 14 insertions(+), 7 deletions(-)
--
2.47.2
--- End Message ---
_______________________________________________
pve-devel mailing list
pve-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel
