[pve-devel] [PATCH ifupdown2] allow address on vlan aware bridge

[Alexandre DERUMIER]

BTW,

They are 2 configs changes needed 

for bond:

currently we use 

 slaves eth0 eth1.


classic ifupdown support also "bond-slaves eth0 eth1" since debian squeeze.

ifupdown2 support only "bond-slaves".



for vlan aware bridge:
 our ifupdown script "bridgevlan", add bridge-vids 2-4094 if it's not defined by default. 

 but ifupdown2 need to be explicitly defined



I don't known if we could update pve-common management to cleanup current config,
or make a post-install script in ifupdown2 to update config ?


Not sure what is the best way ?



----- Mail original -----
De: "Wolfgang Bumiller" <w.bumiller at proxmox.com>
À: "aderumier" <aderumier at odiso.com>
Cc: "pve-devel" <pve-devel at pve.proxmox.com>
Envoyé: Lundi 18 Juin 2018 10:02:51
Objet: Re: [pve-devel] [PATCH ifupdown2] allow address on vlan aware bridge

Do you think upstream might accept this? I find it rather weird that 
this needs patching. I've always been using this kind of setup with 
regular ifupdown. 

On Sat, Jun 16, 2018 at 03:07:23PM +0200, Alexandre Derumier wrote: 
> --- 
> .../0005-allow-address-on-vlan-aware-bridge.patch | 33 ++++++++++++++++++++++ 
> debian/patches/series | 1 + 
> 2 files changed, 34 insertions(+) 
> create mode 100644 debian/patches/pve/0005-allow-address-on-vlan-aware-bridge.patch 
> 
> diff --git a/debian/patches/pve/0005-allow-address-on-vlan-aware-bridge.patch b/debian/patches/pve/0005-allow-address-on-vlan-aware-bridge.patch 
> new file mode 100644 
> index 0000000..5f44911 
> --- /dev/null 
> +++ b/debian/patches/pve/0005-allow-address-on-vlan-aware-bridge.patch 
> @@ -0,0 +1,33 @@ 
> +From e74d34a4e5c85f98683c61558c8fae8453d2b412 Mon Sep 17 00:00:00 2001 
> +From: Alexandre Derumier <aderumier at odiso.com> 
> +Date: Sat, 16 Jun 2018 14:59:46 +0200 
> +Subject: [PATCH] allow address on vlan aware bridge 
> + 
> +This will allow to setup ip address on vlan aware bridge. 
> +Note that in this case, the ip will use the default PVID. 
> +--- 
> + ifupdown2/ifupdown/utils.py | 8 -------- 
> + 1 file changed, 8 deletions(-) 
> + 
> +diff --git a/ifupdown2/ifupdown/utils.py b/ifupdown2/ifupdown/utils.py 
> +index f7a89a7..cee3425 100644 
> +--- a/ifupdown2/ifupdown/utils.py 
> ++++ b/ifupdown2/ifupdown/utils.py 
> +@@ -346,14 +346,6 @@ class utils(): 
> + else: 
> + cls.logger.info(msg) 
> + return False 
> +- elif (ifaceobj.link_kind & ifaceLinkKind.BRIDGE 
> +- and ifaceobj.link_privflags & ifaceLinkPrivFlags.BRIDGE_VLAN_AWARE): 
> +- msg = '%s bridge vlan aware interfaces' 
> +- if syntax_check: 
> +- cls.logger.warning(msg) 
> +- else: 
> +- cls.logger.info(msg) 
> +- return False 
> + return True 
> + 
> + @classmethod 
> +-- 
> +2.11.0 
> + 
> diff --git a/debian/patches/series b/debian/patches/series 
> index 844d2f5..510229f 100644 
> --- a/debian/patches/series 
> +++ b/debian/patches/series 
> @@ -4,3 +4,4 @@ pve/0001-config-tuning.patch 
> pve/0002-manual-interfaces-set-link-up.patch 
> pve/0003-don-t-remove-tap-veth-fwpr-interfaces-from-bridge-on.patch 
> pve/0004-add-dummy-mtu-bridgevlanport-modules.patch 
> +pve/0005-allow-address-on-vlan-aware-bridge.patch 
> -- 
> 2.11.0