When creating a vlan zone and vnet, we get all the bridge interfaces (slaves) and add them to the new vlan bridge. We get all the bridge interfaces using `/sys/class/net` and then filter them using a regex. This regex was missing the new `if` and `nic` prefixes which were introduced with the pve-network-interface-pinning tool.
This is more of a stop-gap, we can probably remove the regex completely as I don't see any reason to filter by "physical" interfaces here. If there is a need, we can still run `ip link` and check the attributes with `PVE::Network::ip_link_is_physical`. Fixes: #6806 Signed-off-by: Gabriel Goller <[email protected]> --- src/PVE/Network/SDN/Zones/Plugin.pm | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/PVE/Network/SDN/Zones/Plugin.pm b/src/PVE/Network/SDN/Zones/Plugin.pm index 83372e703cf5..0a977fd35a13 100644 --- a/src/PVE/Network/SDN/Zones/Plugin.pm +++ b/src/PVE/Network/SDN/Zones/Plugin.pm @@ -371,7 +371,7 @@ sub get_bridge_ifaces { my $dir = "/sys/class/net/$bridge/brif"; PVE::Tools::dir_glob_foreach( $dir, - '(((eth|bond)\d+|en[^.]+)(\.\d+)?)', + '(((eth|bond|nic|if)\d+|en[^.]+)(\.\d+)?)', sub { push @bridge_ifaces, $_[0]; }, -- 2.47.3 _______________________________________________ pve-devel mailing list [email protected] https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel
