[pve-devel] applied: [PATCH proxmox-widget-toolkit] add vlan interface support

[Thomas Lamprecht]

On 1/30/20 6:11 PM, Alexandre DERUMIER wrote:
>>> applied, thanks! Tried to add some note about the different VLAN combinations, 
>>> I hope I made it not even more confusing ^^ 
> 
> I wonder if pve-docs could be improve with some screenshots of network gui ?
> Currently we only have /etc/network/interfaces text config.
> 
> 
> BTW,
> some sdn news:
> 
> I'm going to write sdn documentation soon, as I have some proxmox users currently pinging
> me by mail or on the forum to test it.
> I just need to polish gui a little bit more, and add vnet in ct/vm nic.

Don't let them nag to much ;P But yeah, once that is ready some feedback
we can work with would be great.

> 
> I'm also currently looking to see if I could replace the "uplink" option for vlan/qinq,
> by simply "bridge", like this use could simply create vmbrX on each host, and sdn will use 
> it. (with ou without vlan-aware), and it shouldn't break current setup.
> I'll do tests to see if it's working fine.
> 

OK

> 
> 
> ----- Mail original -----
> De: "Thomas Lamprecht" <t.lamprecht at proxmox.com>
> À: "pve-devel" <pve-devel at pve.proxmox.com>, "aderumier" <aderumier at odiso.com>
> Envoyé: Jeudi 30 Janvier 2020 17:55:00
> Objet: applied: [pve-devel] [PATCH proxmox-widget-toolkit] add vlan interface support
> 
> On 1/28/20 11:24 AM, Alexandre Derumier wrote: 
>> vlan-raw-device && vlan-id field are only 
>> enabled if interface name is different than interfaceX.Y 
>>
>> I have added a listener on iface, 
>> to enable them live if user want a custom name for vlan interface 
>>
>> Signed-off-by: Alexandre Derumier <aderumier at odiso.com> 
>> --- 
>> Toolkit.js | 8 ++++++ 
>> Utils.js | 2 ++ 
>> node/NetworkEdit.js | 67 +++++++++++++++++++++++++++++++++++++++++++-- 
>> node/NetworkView.js | 17 +++++++++++- 
>> 4 files changed, 90 insertions(+), 4 deletions(-) 
>>
> 
> applied, thanks! Tried to add some note about the different VLAN combinations, 
> I hope I made it not even more confusing ^^ 
>