superseded: [PATCH access-control/cluster/manager/network/qemu-server 0/9] fix #7294: pool: add SDN VNets as pool members

[David Riley <d.riley@proxmox.com>]

superseded by:
https://lore.proxmox.com/pve-devel/20260626131035.112374-1-d.riley@proxmox.com/

On 6/11/26 4:59 PM, David Riley wrote:
> This series implements support for adding SDN VNets to resource pools,
> resolving #7294 [0]. This series depends on the v3 'fix #7520: sdn:
> prune orphaned ACLs and handle VNet migrations' [1].
>
> It does not, however, add zones as pool members as requested in #7294.
> Zones currently share ACL paths for managing the zone itself and
> allocating VNets within it. This makes self-service VNet management
> without also granting zone management (and its associated
> side-effects) difficult.
>
> This patch series extends the pool section in the user.cfg and
> introduces a new network property to the pool configuration which will
> hold VNet entries:
> * vnet/<zone>/<vnet>
> * vnet/<zone>/<vnet>/<vlan>
>
> The type prefix allows future extension to other network resource
> types.
>
> To prevent potential data loss from overwriting newly added VNets, a
> cluster-version check is added which ensures all nodes are running a
> version that supports this feature. Note: The hardcoded version guard
> should be updated to match the final target release when being
> applied.
>
> The existing version check helpers were moved from `qemu-server` to a
> new module within `pve-cluster` to make them available for this
> implementation, and any future developments that require gatekeeping.
> Appropriate attribution has been included for the relocated code.
> Please let me know if this organizational move aligns with current
> design preferences or additional adjustments are needed.
>
> [0] https://bugzilla.proxmox.com/show_bug.cgi?id=7294
> [1] https://lore.proxmox.com/pve-devel/20260603145523.120075-1-d.riley@proxmox.com/
>
>
> pve-manager:
>
> David Riley (3):
>    ui: replace var with let to match style guide for variable declaration
>    fix #7294: api: pool: add SDN VNets as pool members
>    fix #7294: ui: pool: add SDN VNets as pool members
>
>   PVE/API2/Pool.pm                 | 137 ++++++++++++++++++++++++++++++-
>   www/css/ext6-pve.css             |  13 +++
>   www/manager6/Utils.js            |   1 +
>   www/manager6/grid/PoolMembers.js | 130 +++++++++++++++++++++++++----
>   4 files changed, 265 insertions(+), 16 deletions(-)
>
>
> pve-access-control:
>
> David Riley (1):
>    fix #7294: acl: pool: add SDN VNets as pool members
>
>   src/PVE/AccessControl.pm  | 88 +++++++++++++++++++++++++++++++++++++--
>   src/PVE/RPCEnvironment.pm | 47 +++++++++++++++++++++
>   src/test/parser_writer.pl | 53 +++++++++++++++++++----
>   3 files changed, 176 insertions(+), 12 deletions(-)
>
>
> pve-network:
>
> David Riley (2):
>    fix #7294: sdn: register api formats for zones and vnets
>    fix #7294: sdn: vnet: update pool members on vnet migration and
>      deletion
>
>   src/PVE/Network/SDN.pm              | 15 +++++++++++++++
>   src/PVE/Network/SDN/VnetPlugin.pm   | 23 ++++++++++++++++++++---
>   src/PVE/Network/SDN/Zones/Plugin.pm | 23 ++++++++++++++++++++---
>   3 files changed, 55 insertions(+), 6 deletions(-)
>
>
> pve-cluster:
>
> David Riley (2):
>    cluster: add helpers module with version comparison functions
>    fix #7294: cluster: helpers: add cluster-wide version assertion
>
>   debian/pve-cluster.install |  1 +
>   src/PVE/Cluster/Helpers.pm | 86 ++++++++++++++++++++++++++++++++++++++
>   src/PVE/Cluster/Makefile   |  2 +-
>   3 files changed, 88 insertions(+), 1 deletion(-)
>   create mode 100644 src/PVE/Cluster/Helpers.pm
>
>
> qemu-server:
>
> David Riley (1):
>    fix #7294: helpers: use cluster-wide version helper
>
>   src/PVE/QemuMigrate.pm        |  2 +-
>   src/PVE/QemuServer/Helpers.pm | 40 +----------------------------------
>   2 files changed, 2 insertions(+), 40 deletions(-)
>
>
> Summary over all repositories:
>    15 files changed, 586 insertions(+), 75 deletions(-)
>