diff options
| author | Christian Poessinger <christian@poessinger.com> | 2022-12-17 08:32:44 +0100 |
|---|---|---|
| committer | Christian Poessinger <christian@poessinger.com> | 2022-12-17 08:32:47 +0100 |
| commit | fe80ae5533bebb863a39861a2c27a0b07f84845e (patch) | |
| tree | 16e7d14ea46e9a721d845f8651b6cbe094e61c0b /src | |
| parent | 8637b02bb60baf499fc63696c28aa1af3559c308 (diff) | |
| download | vyos-1x-fe80ae5533bebb863a39861a2c27a0b07f84845e.tar.gz vyos-1x-fe80ae5533bebb863a39861a2c27a0b07f84845e.zip | |
bonding: T4878: use more is_node_changed() over leaf_node_changed()
The implementation of is_node_changed() is less error prone and should always
be favoured.
Diffstat (limited to 'src')
| -rwxr-xr-x | src/conf_mode/interfaces-bonding.py | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/src/conf_mode/interfaces-bonding.py b/src/conf_mode/interfaces-bonding.py index b883ebef2..9936620c8 100755 --- a/src/conf_mode/interfaces-bonding.py +++ b/src/conf_mode/interfaces-bonding.py @@ -21,6 +21,7 @@ from netifaces import interfaces from vyos.config import Config from vyos.configdict import get_interface_dict +from vyos.configdict import is_node_changed from vyos.configdict import leaf_node_changed from vyos.configdict import is_member from vyos.configdict import is_source_interface @@ -81,10 +82,10 @@ def get_config(config=None): if 'mode' in bond: bond['mode'] = get_bond_mode(bond['mode']) - tmp = leaf_node_changed(conf, base + [ifname, 'mode']) + tmp = is_node_changed(conf, base + [ifname, 'mode']) if tmp: bond['shutdown_required'] = {} - tmp = leaf_node_changed(conf, base + [ifname, 'lacp-rate']) + tmp = is_node_changed(conf, base + [ifname, 'lacp-rate']) if tmp: bond['shutdown_required'] = {} # determine which members have been removed |