diff options
| author | Christian Poessinger <christian@poessinger.com> | 2022-12-30 09:47:04 +0100 |
|---|---|---|
| committer | Christian Poessinger <christian@poessinger.com> | 2022-12-30 09:49:03 +0100 |
| commit | c42736469a06851a49d4ea5c12c57ecac238b7a4 (patch) | |
| tree | 6f78761efbdfda8d3092f39b5157ba16583c5aef | |
| parent | b8899246cda23a999f35e1c102b30c841623e0ca (diff) | |
| download | vyos-1x-c42736469a06851a49d4ea5c12c57ecac238b7a4.tar.gz vyos-1x-c42736469a06851a49d4ea5c12c57ecac238b7a4.zip | |
tunnel: T4391: use is_node_changed() over error prone leaf_node_changed()
We only need to use leaf_node_changed() if we are interested in the detailed
change to a CLI node (what was the previous content). If we are only interested
in if a node changed "at all" is_node_changed() should be used.
| -rwxr-xr-x | src/conf_mode/interfaces-tunnel.py | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/conf_mode/interfaces-tunnel.py b/src/conf_mode/interfaces-tunnel.py index acef1fda7..e2701d9d3 100755 --- a/src/conf_mode/interfaces-tunnel.py +++ b/src/conf_mode/interfaces-tunnel.py @@ -21,7 +21,7 @@ from netifaces import interfaces from vyos.config import Config from vyos.configdict import get_interface_dict -from vyos.configdict import leaf_node_changed +from vyos.configdict import is_node_changed from vyos.configverify import verify_address from vyos.configverify import verify_bridge_delete from vyos.configverify import verify_interface_exists @@ -52,7 +52,7 @@ def get_config(config=None): ifname, tunnel = get_interface_dict(conf, base) if 'deleted' not in tunnel: - tmp = leaf_node_changed(conf, base + [ifname, 'encapsulation']) + tmp = is_node_changed(conf, base + [ifname, 'encapsulation']) if tmp: tunnel.update({'encapsulation_changed': {}}) # We also need to inspect other configured tunnels as there are Kernel |