From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from gate001.proxmox.com (gate001.proxmox.com [45.144.208.40]) by lore.proxmox.com (Postfix) with ESMTPS id 7D3A81FF0B4 for ; Wed, 09 Sep 2026 09:53:47 +0200 (CEST) Received: from gate001.proxmox.com (localhost.localdomain [127.0.0.1]) by gate001.proxmox.com (Proxmox) with ESMTP id D54452159E; Wed, 09 Sep 2026 09:53:42 +0200 (CEST) From: Hannes Laimer To: pve-devel@lists.proxmox.com Subject: [PATCH pve-network] sdn: zones: look the underlay interface up correctly Date: Wed, 9 Sep 2026 09:53:28 +0200 Message-ID: <20260909075328.726069-1-h.laimer@proxmox.com> X-Mailer: git-send-email 2.47.3 MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Bm-Milter-Handled: 55990f41-d878-4baa-be0a-ee34c49e34d2 X-Bm-Transport-Timestamp: 1788940406100 X-SPAM-LEVEL: Spam detection results: 0 AWL 0.586 Adjusted score from AWL reputation of From: address DMARC_MISSING 0.1 Missing DMARC policy KAM_DMARC_STATUS 0.01 Test Rule for DKIM or SPF Failure with Strict Alignment (newer systems) RCVD_IN_DNSWL_MED -2.3 Sender listed at https://www.dnswl.org/, medium trust SPF_HELO_NONE 0.001 SPF: HELO does not publish an SPF Record SPF_PASS -0.001 SPF: sender matches SPF record Message-ID-Hash: W6A7BKS73BB5KEAW3F3UX7LHJONVJOL2 X-Message-ID-Hash: W6A7BKS73BB5KEAW3F3UX7LHJONVJOL2 X-MailFrom: h.laimer@proxmox.com X-Mailman-Rule-Misses: dmarc-mitigation; no-senders; approved; loop; banned-address; emergency; member-moderation; nonmember-moderation; administrivia; implicit-dest; max-recipients; max-size; news-moderation; no-subject; digests; suspicious-header X-Mailman-Version: 3.3.10 Precedence: list List-Id: Proxmox VE development discussion List-Help: List-Owner: List-Post: List-Subscribe: List-Unsubscribe: The interfaces reader returns the table under a key, and the vxlan and evpn zones looked their underlay interface up at the top level of the reader's result, so the MTU they derive from it never applied and every vnet bridge of theirs got the default or the zone's own MTU. Signed-off-by: Hannes Laimer --- the alternative would be already passing `->{ifaces}`, but we'd lose the options like that. i think this is better. src/PVE/Network/SDN/Zones/EvpnPlugin.pm | 3 ++- src/PVE/Network/SDN/Zones/VxlanPlugin.pm | 3 ++- 2 files changed, 4 insertions(+), 2 deletions(-) diff --git a/src/PVE/Network/SDN/Zones/EvpnPlugin.pm b/src/PVE/Network/SDN/Zones/EvpnPlugin.pm index 0e79707..f89decf 100644 --- a/src/PVE/Network/SDN/Zones/EvpnPlugin.pm +++ b/src/PVE/Network/SDN/Zones/EvpnPlugin.pm @@ -214,7 +214,8 @@ sub generate_sdn_config { my $mtu = 1450; if ($iface) { - $mtu = $interfaces_config->{$iface}->{mtu} - 50 if $interfaces_config->{$iface}->{mtu}; + $mtu = $interfaces_config->{ifaces}->{$iface}->{mtu} - 50 + if $interfaces_config->{ifaces}->{$iface}->{mtu}; } $mtu = $plugin_config->{mtu} if $plugin_config->{mtu}; diff --git a/src/PVE/Network/SDN/Zones/VxlanPlugin.pm b/src/PVE/Network/SDN/Zones/VxlanPlugin.pm index a408261..b4e743e 100644 --- a/src/PVE/Network/SDN/Zones/VxlanPlugin.pm +++ b/src/PVE/Network/SDN/Zones/VxlanPlugin.pm @@ -119,7 +119,8 @@ sub generate_sdn_config { my $mtu = 1450; if ($iface) { - $mtu = $interfaces_config->{$iface}->{mtu} - 50 if $interfaces_config->{$iface}->{mtu}; + $mtu = $interfaces_config->{ifaces}->{$iface}->{mtu} - 50 + if $interfaces_config->{ifaces}->{$iface}->{mtu}; } $mtu = $plugin_config->{mtu} if $plugin_config->{mtu}; -- 2.47.3