diff options
| author | John Estabrook <jestabro@vyos.io> | 2024-08-08 14:24:00 -0500 |
|---|---|---|
| committer | Mergify <37929162+mergify[bot]@users.noreply.github.com> | 2024-08-13 00:43:25 +0000 |
| commit | a6362cc32135b39b4320221006f48fef78f40c24 (patch) | |
| tree | 82a6fd7325f2b0314d2c8a411fc8520d7d587a55 /src/conf_mode/firewall.py | |
| parent | e1d5cb8b56a6b103afa7326d94291e64bd2eba47 (diff) | |
| download | vyos-1x-mergify/bp/sagitta/pr-3961.tar.gz vyos-1x-mergify/bp/sagitta/pr-3961.zip | |
configverify: T6642: verify_interface_exists requires config_dict argmergify/bp/sagitta/pr-3961
The function verify_interface_exists requires a reference to the ambient
config_dict rather than creating an instance. As access is required to
the 'interfaces' path, provide as attribute of class ConfigDict, so as
not to confuse path searches of script-specific config_dict instances.
(cherry picked from commit 5f23b7275564cfaa7c178d320868b5f5e86ae606)
Diffstat (limited to 'src/conf_mode/firewall.py')
| -rwxr-xr-x | src/conf_mode/firewall.py | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/conf_mode/firewall.py b/src/conf_mode/firewall.py index e96e57154..9e95d7794 100755 --- a/src/conf_mode/firewall.py +++ b/src/conf_mode/firewall.py @@ -335,7 +335,7 @@ def verify(firewall): raise ConfigError(f'Flowtable "{flowtable}" requires at least one interface') for ifname in flowtable_conf['interface']: - verify_interface_exists(ifname) + verify_interface_exists(firewall, ifname) if dict_search_args(flowtable_conf, 'offload') == 'hardware': interfaces = flowtable_conf['interface'] |