diff options
| author | Christian Poessinger <christian@poessinger.com> | 2019-09-19 21:26:40 +0200 |
|---|---|---|
| committer | Christian Poessinger <christian@poessinger.com> | 2019-09-19 22:22:24 +0200 |
| commit | 0387cae19c28d1d5b8ccf45596841ba44a2eefc9 (patch) | |
| tree | dcb6328ae782da6d333d6d224fe79d1832271087 | |
| parent | eb9c6ff745fc5d4e23c224a441874ae6fcf97ac5 (diff) | |
| download | vyos-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-x | src/conf_mode/interface-bonding.py | 5 |
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 ' \ |