When using non-vlan-aware bridges for the VLAN or QinQ zones, the generated SDN ifupdown2 configuration uses the physical NIC as port on the generated vnet bridge, since it is not possible to create a VLAN subinterface directly on the bridge.
This causes issues when pinning NIC names, after a VLAN or QinQ zone has already been created on a non-vlan-aware zone. The name of the physical interface changes after a reboot, but the generated SDN configuration doesn't. Avoid this by detecting any VLAN / QinQ zone that uses a non-vlan-aware bridge and regenerate the SDN configuration in that case. This should also fix cases where the network interface gets renamed for other reasons (e.g. not pinned network interfaces and updates to the kernel). Signed-off-by: Stefan Hanreich <[email protected]> --- bin/pve-sdn-commit | 22 ++++++++++++++++++++++ 1 file changed, 22 insertions(+) diff --git a/bin/pve-sdn-commit b/bin/pve-sdn-commit index 6eeba301c..aa7e9b290 100644 --- a/bin/pve-sdn-commit +++ b/bin/pve-sdn-commit @@ -6,6 +6,7 @@ use warnings; use Time::HiRes qw(usleep); use PVE::Cluster; +use PVE::IPRoute2; use PVE::Network::SDN; use PVE::Network::SDN::Zones; use PVE::Network::SDN::Vnets; @@ -53,6 +54,17 @@ sub fabrics_changed { return has_pending_changes($pending_fabrics) || has_pending_changes($pending_nodes); } +sub zone_uses_non_vlan_aware_bridge { + my ($zone, $ip_links) = @_; + + return 0 if ($zone->{type} ne 'vlan' && $zone->{type} ne 'qinq'); + + my $ip_link = $ip_links->{ $zone->{bridge} }; + return 0 if !defined($ip_link); + + return !PVE::IPRoute2::bridge_is_vlan_aware($ip_link); +} + sub sdn_changed { my $running_config = PVE::Network::SDN::running_config(); @@ -71,6 +83,16 @@ sub sdn_changed { return 1 if has_pending_changes($pending_config); } + my $ip_links = PVE::IPRoute2::ip_link_details(); + + for my $zone (values $configs->{zones}->{ids}->%*) { + return 1 if zone_uses_non_vlan_aware_bridge($zone, $ip_links); + } + + for my $running_zone (values $running_config->{zones}->{ids}->%*) { + return 1 if zone_uses_non_vlan_aware_bridge($running_zone, $ip_links); + } + return fabrics_changed(); } -- 2.47.3 _______________________________________________ pve-devel mailing list [email protected] https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel
