]> git.proxmox.com Git - mirror_ifupdown2.git/commit
addons: vrf: bring up master on first slave ifup if WITH_DEPENDS is set
authorRoopa Prabhu <roopa@cumulusnetworks.com>
Wed, 30 Nov 2016 21:06:36 +0000 (13:06 -0800)
committerRoopa Prabhu <roopa@cumulusnetworks.com>
Wed, 30 Nov 2016 22:02:19 +0000 (14:02 -0800)
commit1ce8b5bb82801c516e825a6eccb51bc4e5c917dd
treeaa066f90454cfe3070ca3c00fc8ea7abeadddfa0
parent5e263869214607f0ba3b33b3cfdbfc799032b1fd
addons: vrf: bring up master on first slave ifup if WITH_DEPENDS is set

Ticket: CM-12988
Reviewed By: julien, nikhil
Testing Done: Tested ifdown and ifup of vrf device with --with-depends

This patch fixes transient errors like below on vrf slaves when
vrf device is being brought up with --with-depends:
"error: swp1: vrf vrf1 not around, skipping vrf config
error: br100: vrf vrf1 not around, skipping vrf config
error: br101: vrf vrf1 not around, skipping vrf config"

In this patch, the vrf device is brought up on bringing
up of the first vrf slave. This is also done in the normal
ifreload -a case.

history on --with-depends for vrf: On vrf device down,
a bunch of slave state gets cleaned up.
and today, ifup of vrf device alone does not fix all that state
especially when there are vrr (macvlan) interfaces involved.
One has to use --with-depends. This is now also part of documentation
https://tickets.cumulusnetworks.com/browse/UD-851

$ifdown vrf1

$ifup vrf1 --with-depends

Signed-off-by: Roopa Prabhu <roopa@cumulusnetworks.com>
addons/vrf.py