This is a breaking API change for zones, vnets and subnets! These previously expected the *complete* config, which is not how our usual SectionConfigs work.
Further, they advertised a 'delete' API parameter which was simply passed through as a config property which of course failed validation. Instead, they now merge the provided changes the way we normally do, and handle the 'delete' parameter. This affects: - /cluster/sdn/zones/<id> - /cluster/sdn/vnets/<id> - /cluster/sdn/vnets/<id>/subnets/<subnet> Additionally, these already took partial updates, but ignored the 'delete' parameter, which is fixed in this series: - /cluster/sdn/ipams/<id> - /cluster/sdn/dns/<id> - /cluster/sdn/controllers/<id> Wolfgang Bumiller (7): Revert "sdn: require ipam in simple plugin for dhcp" api: take partial configs for PUT /cluster/sdn/zones/<id> api: take partial configs for PUT /cluster/sdn/vnets/<id> api: take partial configs for PUT /cluster/sdn/vnets/<n>/subnets/<i> api: handle delete parameter when updating ipams api: handle delete parameter when updating dns entries api: handle delete parameter when updating controllers src/PVE/API2/Network/SDN/Controllers.pm | 7 +++++++ src/PVE/API2/Network/SDN/Dns.pm | 7 +++++++ src/PVE/API2/Network/SDN/Ipams.pm | 7 +++++++ src/PVE/API2/Network/SDN/Subnets.pm | 12 ++++++++++-- src/PVE/API2/Network/SDN/Vnets.pm | 24 +++++++++++++++++++---- src/PVE/API2/Network/SDN/Zones.pm | 18 ++++++++++++++--- src/PVE/Network/SDN/Zones/SimplePlugin.pm | 1 - 7 files changed, 66 insertions(+), 10 deletions(-) -- 2.39.2 _______________________________________________ pve-devel mailing list pve-devel@lists.proxmox.com https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel
