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

addons: vrf: removing link-down after slave is removed from a VRF

This was added back when ifupdown2 was using brctl to del bridges.
brctl doesn't allow deleting a bridge if the bridge is still admin up.
Now that we moved to netlink this is not needed anymore.

Signed-off-by: Julien Fortin <julien@cumulusnetworks.com>
This commit is contained in:
Julien Fortin
2020-06-26 14:38:08 +02:00
parent f343240c18
commit 366da8b334

View File

@@ -1009,12 +1009,6 @@ class vrf(Addon, moduleBase):
try:
self._handle_existing_connections(ifaceobj, vrfname)
self.netlink.link_set_nomaster(ifacename)
# Down this slave only if it is a slave ifupdown2 manages.
# we dont want to down slaves that maybe up'ed by
# somebody else. One such example is a macvlan device
# which ifupdown2 addressvirtual addon module auto creates
if ifaceobj:
self.netlink.link_down(ifacename)
except Exception as e:
self.logger.warning('%s: %s' %(ifacename, str(e)))