summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorChristian Poessinger <christian@poessinger.com>2019-09-19 21:26:40 +0200
committerChristian Poessinger <christian@poessinger.com>2019-09-19 22:22:24 +0200
commit0387cae19c28d1d5b8ccf45596841ba44a2eefc9 (patch)
treedcb6328ae782da6d333d6d224fe79d1832271087
parenteb9c6ff745fc5d4e23c224a441874ae6fcf97ac5 (diff)
downloadvyos-1x-0387cae19c28d1d5b8ccf45596841ba44a2eefc9.tar.gz
vyos-1x-0387cae19c28d1d5b8ccf45596841ba44a2eefc9.zip
bonding: T1614: allow adding disabled interfaces to bond
An interface can only be added in disabled state to a bond (ensured via ifconfig.py). Also interfaces can be disabled during runtime in a bond which is supported by the Linux Kernel - so why should be add a restriction here? makes no sense.
-rwxr-xr-xsrc/conf_mode/interface-bonding.py5
1 files changed, 0 insertions, 5 deletions
diff --git a/src/conf_mode/interface-bonding.py b/src/conf_mode/interface-bonding.py
index f0a33beff..ac3e1b867 100755
--- a/src/conf_mode/interface-bonding.py
+++ b/src/conf_mode/interface-bonding.py
@@ -279,11 +279,6 @@ def verify(bond):
raise ConfigError('can not enslave interface {} which already ' \
'belongs to {}'.format(intf, tmp))
- # we can not add disabled slave interfaces to our bond
- if conf.exists('interfaces ethernet ' + intf + ' disable'):
- raise ConfigError('can not enslave disabled interface {}' \
- .format(intf))
-
# can not add interfaces with an assigned address to a bond
if conf.exists('interfaces ethernet ' + intf + ' address'):
raise ConfigError('can not enslave interface {} which has an address ' \