diff options
author | Christian Breunig <christian@breunig.cc> | 2023-12-24 10:17:20 +0100 |
---|---|---|
committer | Christian Breunig <christian@breunig.cc> | 2023-12-24 13:13:27 +0100 |
commit | 5e7a8288d06a6d6beee5e1abd2e06698ab778650 (patch) | |
tree | d748b757485782baab33f06131f88c43e01d2222 /src/conf_mode/firewall.py | |
parent | 89cd75b8dbe5cc145a4423bf10faa76fd6bdcdbf (diff) | |
download | vyos-1x-5e7a8288d06a6d6beee5e1abd2e06698ab778650.tar.gz vyos-1x-5e7a8288d06a6d6beee5e1abd2e06698ab778650.zip |
T5837: cleanup use of calls to vyos.configdict.node_changed()
node_changed() will return a list of changed keys under "path". We are not
always interested what changed, sometimes we are only interested if something
changed at all, that what vyos.configdict.is_node_changed() is for.
Diffstat (limited to 'src/conf_mode/firewall.py')
-rwxr-xr-x | src/conf_mode/firewall.py | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/conf_mode/firewall.py b/src/conf_mode/firewall.py index ceed0cf31..da6724fde 100755 --- a/src/conf_mode/firewall.py +++ b/src/conf_mode/firewall.py @@ -23,7 +23,7 @@ from sys import exit from vyos.base import Warning from vyos.config import Config -from vyos.configdict import node_changed +from vyos.configdict import is_node_changed from vyos.configdiff import get_config_diff, Diff from vyos.configdep import set_dependents, call_dependents from vyos.configverify import verify_interface_exists @@ -133,7 +133,7 @@ def get_config(config=None): with_recursive_defaults=True) - firewall['group_resync'] = bool('group' in firewall or node_changed(conf, base + ['group'])) + firewall['group_resync'] = bool('group' in firewall or is_node_changed(conf, base + ['group'])) if firewall['group_resync']: # Update nat and policy-route as firewall groups were updated set_dependents('group_resync', conf) |