diff options
| author | zsdc <taras@vyos.io> | 2022-12-15 11:13:02 +0200 |
|---|---|---|
| committer | zsdc <taras@vyos.io> | 2022-12-15 11:13:02 +0200 |
| commit | 837357071135b87532796dfab88d53ae7cba9e45 (patch) | |
| tree | 7f63f1fbebf27fadaa00da119b92b7799907273e | |
| parent | 4c10181bf2b162a2cd703157eaf4f72d6c7c4a71 (diff) | |
| download | vyos-1x-837357071135b87532796dfab88d53ae7cba9e45.tar.gz vyos-1x-837357071135b87532796dfab88d53ae7cba9e45.zip | |
bonding: T4878: Fixed unnecessary bonding flapping during commit
There was a mistake in a config level that caused triggering the
`shutdown_required` flag, even if there were no new interfaces added to a
bonding.
This commit sets the proper config level to avoid the problem.
| -rwxr-xr-x | src/conf_mode/interfaces-bonding.py | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/conf_mode/interfaces-bonding.py b/src/conf_mode/interfaces-bonding.py index 21cf204fc..b883ebef2 100755 --- a/src/conf_mode/interfaces-bonding.py +++ b/src/conf_mode/interfaces-bonding.py @@ -116,7 +116,7 @@ def get_config(config=None): if dict_search('member.interface', bond): for interface, interface_config in bond['member']['interface'].items(): # Check if member interface is a new member - if not conf.exists_effective(['member', 'interface', interface]): + if not conf.exists_effective(base + [ifname, 'member', 'interface', interface]): bond['shutdown_required'] = {} # Check if member interface is disabled |