Re: [pve-devel] [PATCH v4 pve-network 10/33] api: add endpoints for managing PVE IPAM

[Stefan Hanreich]

On 11/20/23 13:34, Stefan Hanreich wrote:
> Not sure about this, since the endpoint returns the state of the PVE
> IPAM and never returns the state of Netbox IPAM, for instance. Since if
> you want to inspect that state you would use the Netbox API / Web UI.
> For that reason it would make sense for me to use pve in the API path.
> 
>> So, I think that theses should be subpath of subnets
>>
>> /subnets/<subnetid/ips
> 
> This sounds like a good idea - and is probably better than what I
> proposed. I'll quickly talk with Thomas about this.

What would you think about adding it to /vnets instead of subnets?

Since the API endpoints are calling the VNets functions - which are
mostly calling the Subnet-Functions, yes - but they require less
parameters and are therefore easier to handle because we don't have to
send too much (implicitly available) information to/from the frontend.


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