mirror/CumulusNetworks-ifupdown2
1
0
Fork 0
mirror of https://github.com/CumulusNetworks/ifupdown2.git synced 2024-05-06 15:54:50 +00:00
Code

addons: addressvirtual: allowing address-virtual attr for vrf slave interfaces

Browse Source 
Ticket: CM-11745
Reviewed By: Roopa, Nikhil G
Testing Done: configuration provided in the bug

User was attempting to configure a traditional bridge under a VRF with VRR.
When issuing ifreload -a, the configuration failed to apply with an error.

Applying an address-virtual keyword to an interface with upper interfaces
or parent interfaces was not allowed. But now we are allowing the use of
this keyword only for vrf slaves.

Signed-off-by: Julien Fortin <julien@cumulusnetworks.com>
This commit is contained in:
Julien Fortin
2016-07-13 23:25:07 +01:00
parent eb377c6ca2
commit a28b54359f
1 changed files with 2 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
addons/addressvirtual.py
View File

@ -283,7 +283,8 @@ class addressvirtual(moduleBase):
self._remove_address_config(ifaceobj, address_virtual_list)
return
if ifaceobj.upperifaces:
if ifaceobj.upperifaces and \
not ifaceobj.link_privflags & ifaceLinkPrivFlags.VRF_SLAVE:
self.log_error('%s: invalid placement of address-virtual lines (must be configured under an interface with no upper interfaces or parent interfaces)'
% (ifaceobj.name), ifaceobj)
return