[pve-devel] [PATCH] add vlan aware bridge ifupdown script v2

[Alexandre DERUMIER]

I don't known if we could contribute to ifupdown2  project ? (But I'm pretty poor in python)
It's almost working on jessie (I have 1 or 2 patches, because cumulus have sometime different path for binaries).

big missing part is openvswitch, and tunnel interfaces



----- Mail original -----
De: "aderumier" <aderumier at odiso.com>
À: "Wolfgang Bumiller" <w.bumiller at proxmox.com>
Cc: "pve-devel" <pve-devel at pve.proxmox.com>
Envoyé: Mardi 28 Juillet 2015 09:57:50
Objet: Re: [pve-devel] [PATCH] add vlan aware bridge ifupdown script v2

>> + if [[ $port =~ ^(eth|bond|wlan)[0-9]{1,2}$ ]] 


>>Isn't there a better way to do this? 
>>Like checking the interface type via /proc? Because otherwise we end up 
>>with more threads like: 
>><http://forum.proxmox.com/threads/22504-Unable-to-see-Infiniband-interfaces-in-Proxmox> 
>> 
>>Restricting interfaces to names has always been problematic. 
>>We already have a problem when someone installes pve packages from a 
>>normal debian distribution after enabling systemd's interface naming 
>>scheme. 

yes, I think also it's really stupid to restrict to specific name. 

ifupdown2 allow any name for vlan interface (*.*) for example 
https://github.com/CumulusNetworks/ifupdown2/blob/master/ifupdown2/addons/vlan.py 





----- 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é: Mardi 28 Juillet 2015 09:38:25 
Objet: Re: [pve-devel] [PATCH] add vlan aware bridge ifupdown script v2 

> + if [[ $port =~ ^(eth|bond|wlan)[0-9]{1,2}$ ]] 

Isn't there a better way to do this? 
Like checking the interface type via /proc? Because otherwise we end up 
with more threads like: 
<http://forum.proxmox.com/threads/22504-Unable-to-see-Infiniband-interfaces-in-Proxmox> 

Restricting interfaces to names has always been problematic. 
We already have a problem when someone installes pve packages from a 
normal debian distribution after enabling systemd's interface naming 
scheme. 
_______________________________________________ 
pve-devel mailing list 
pve-devel at pve.proxmox.com 
http://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel