From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from firstgate.proxmox.com (firstgate.proxmox.com [212.224.123.68]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits)) (No client certificate requested) by lists.proxmox.com (Postfix) with ESMTPS id E851377FF6 for ; Thu, 29 Apr 2021 13:18:02 +0200 (CEST) Received: from firstgate.proxmox.com (localhost [127.0.0.1]) by firstgate.proxmox.com (Proxmox) with ESMTP id E5C6D1B2F9 for ; Thu, 29 Apr 2021 13:18:02 +0200 (CEST) Received: from kvmformation3.odiso.net (globalOdiso.M6Lille.odiso.net [89.248.211.242]) by firstgate.proxmox.com (Proxmox) with ESMTP id 60F541B2D5 for ; Thu, 29 Apr 2021 13:18:01 +0200 (CEST) Received: by kvmformation3.odiso.net (Postfix, from userid 0) id 1223875A2F; Thu, 29 Apr 2021 13:18:01 +0200 (CEST) From: Alexandre Derumier To: pve-devel@lists.proxmox.com Date: Thu, 29 Apr 2021 13:17:58 +0200 Message-Id: <20210429111759.231197-3-aderumier@odiso.com> X-Mailer: git-send-email 2.20.1 In-Reply-To: <20210429111759.231197-1-aderumier@odiso.com> References: <20210429111759.231197-1-aderumier@odiso.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-SPAM-LEVEL: Spam detection results: 1 AWL -0.163 Adjusted score from AWL reputation of From: address HEADER_FROM_DIFFERENT_DOMAINS 0.25 From and EnvelopeFrom 2nd level mail domains are different KAM_DMARC_STATUS 0.01 Test Rule for DKIM or SPF Failure with Strict Alignment KAM_LAZY_DOMAIN_SECURITY 1 Sending domain does not have any anti-forgery methods KHOP_HELO_FCRDNS 0.399 Relay HELO differs from its IP's reverse DNS NO_DNS_FOR_FROM 0.379 Envelope sender has no MX or A DNS records SPF_HELO_NONE 0.001 SPF: HELO does not publish an SPF Record SPF_NONE 0.001 SPF: sender does not publish an SPF Record URIBL_BLOCKED 0.001 ADMINISTRATOR NOTICE: The query to URIBL was blocked. See http://wiki.apache.org/spamassassin/DnsBlocklists#dnsbl-block for more information. [qinqplugin.pm] Subject: [pve-devel] [PATCH v2 pve-network 2/3] zones: qinq: classic bridge: fix bridge zone name with multiple zones/vnets X-BeenThere: pve-devel@lists.proxmox.com X-Mailman-Version: 2.1.29 Precedence: list List-Id: Proxmox VE development discussion List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 29 Apr 2021 11:18:02 -0000 Signed-off-by: Alexandre Derumier --- PVE/Network/SDN/Zones/QinQPlugin.pm | 2 +- .../zones/qinq/bridge/expected_sdn_interfaces | 33 +++++++++++++++++-- test/zones/qinq/bridge/sdn_config | 7 +++- .../expected_sdn_interfaces | 6 ++-- .../expected_sdn_interfaces | 6 ++-- 5 files changed, 43 insertions(+), 11 deletions(-) diff --git a/PVE/Network/SDN/Zones/QinQPlugin.pm b/PVE/Network/SDN/Zones/QinQPlugin.pm index 1543fc1..c5016f5 100644 --- a/PVE/Network/SDN/Zones/QinQPlugin.pm +++ b/PVE/Network/SDN/Zones/QinQPlugin.pm @@ -132,7 +132,7 @@ sub generate_sdn_config { foreach my $bridge_iface (@bridge_ifaces) { # use named vlan interface to avoid too long names - my $svlan_iface = "sv_$vnetid"; + my $svlan_iface = "sv_$zoneid"; my $cvlan_iface = "cv_$vnetid"; #svlan diff --git a/test/zones/qinq/bridge/expected_sdn_interfaces b/test/zones/qinq/bridge/expected_sdn_interfaces index 73db5b3..91ef667 100644 --- a/test/zones/qinq/bridge/expected_sdn_interfaces +++ b/test/zones/qinq/bridge/expected_sdn_interfaces @@ -2,7 +2,17 @@ auto cv_myvnet iface cv_myvnet - vlan-raw-device sv_myvnet + vlan-raw-device sv_myzone + vlan-id 100 + +auto cv_myvnet2 +iface cv_myvnet2 + vlan-raw-device sv_myzone + vlan-id 101 + +auto cv_myvnet3 +iface cv_myvnet3 + vlan-raw-device sv_myzone2 vlan-id 100 auto myvnet @@ -11,7 +21,24 @@ iface myvnet bridge_stp off bridge_fd 0 -auto sv_myvnet -iface sv_myvnet +auto myvnet2 +iface myvnet2 + bridge_ports cv_myvnet2 + bridge_stp off + bridge_fd 0 + +auto myvnet3 +iface myvnet3 + bridge_ports cv_myvnet3 + bridge_stp off + bridge_fd 0 + +auto sv_myzone +iface sv_myzone vlan-raw-device eth0 vlan-id 10 + +auto sv_myzone2 +iface sv_myzone2 + vlan-raw-device eth0 + vlan-id 20 diff --git a/test/zones/qinq/bridge/sdn_config b/test/zones/qinq/bridge/sdn_config index 1708d4c..6321603 100644 --- a/test/zones/qinq/bridge/sdn_config +++ b/test/zones/qinq/bridge/sdn_config @@ -3,9 +3,14 @@ vnets => { ids => { myvnet => { tag => 100, type => "vnet", zone => "myzone" }, + myvnet2 => { tag => 101, type => "vnet", zone => "myzone" }, + myvnet3 => { tag => 100, type => "vnet", zone => "myzone2" }, }, }, zones => { - ids => { myzone => { bridge => "vmbr0", tag => 10, ipam => "pve", type => "qinq" } }, + ids => { + myzone => { bridge => "vmbr0", tag => 10, ipam => "pve", type => "qinq" }, + myzone2 => { bridge => "vmbr0", tag => 20, ipam => "pve", type => "qinq" }, + }, }, } diff --git a/test/zones/qinq/bridge_vlanawarevnet/expected_sdn_interfaces b/test/zones/qinq/bridge_vlanawarevnet/expected_sdn_interfaces index ab7572e..59265fd 100644 --- a/test/zones/qinq/bridge_vlanawarevnet/expected_sdn_interfaces +++ b/test/zones/qinq/bridge_vlanawarevnet/expected_sdn_interfaces @@ -2,7 +2,7 @@ auto cv_myvnet iface cv_myvnet - vlan-raw-device sv_myvnet + vlan-raw-device sv_myzone vlan-id 100 auto myvnet @@ -13,7 +13,7 @@ iface myvnet bridge-vlan-aware yes bridge-vids 2-4094 -auto sv_myvnet -iface sv_myvnet +auto sv_myzone +iface sv_myzone vlan-raw-device eth0 vlan-id 10 diff --git a/test/zones/qinq/bridge_vlanprotocol/expected_sdn_interfaces b/test/zones/qinq/bridge_vlanprotocol/expected_sdn_interfaces index 460f1a5..528ceaa 100644 --- a/test/zones/qinq/bridge_vlanprotocol/expected_sdn_interfaces +++ b/test/zones/qinq/bridge_vlanprotocol/expected_sdn_interfaces @@ -2,7 +2,7 @@ auto cv_myvnet iface cv_myvnet - vlan-raw-device sv_myvnet + vlan-raw-device sv_myzone vlan-id 100 auto myvnet @@ -11,8 +11,8 @@ iface myvnet bridge_stp off bridge_fd 0 -auto sv_myvnet -iface sv_myvnet +auto sv_myzone +iface sv_myzone vlan-raw-device eth0 vlan-id 10 vlan-protocol 802.1ad -- 2.20.1