summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJohn Estabrook <jestabro@vyos.io>2024-07-14 16:31:58 -0500
committerMergify <37929162+mergify[bot]@users.noreply.github.com>2024-07-17 13:04:08 +0000
commit70dbc0109282576875e20503a36c99da1073f3b4 (patch)
treee768ef8f1a1bfb585e1adc1cb693e53f0695b8c9
parent71f4d7c721d4b9ebfd005e8429f1d6f5c907a5ff (diff)
downloadvyos-1x-70dbc0109282576875e20503a36c99da1073f3b4.tar.gz
vyos-1x-70dbc0109282576875e20503a36c99da1073f3b4.zip
configdep: T6559: use single dependency list with reset under configd
(cherry picked from commit 52d08b1ec5b2943744daac7123e35fd415f85db2)
-rw-r--r--python/vyos/config.py1
-rw-r--r--python/vyos/configdep.py18
-rwxr-xr-xsrc/services/vyos-configd1
3 files changed, 10 insertions, 10 deletions
diff --git a/python/vyos/config.py b/python/vyos/config.py
index cca65f0eb..b7ee606a9 100644
--- a/python/vyos/config.py
+++ b/python/vyos/config.py
@@ -140,6 +140,7 @@ class Config(object):
self._level = []
self._dict_cache = {}
+ self.dependency_list = []
(self._running_config,
self._session_config) = self._config_source.get_configtree_tuple()
diff --git a/python/vyos/configdep.py b/python/vyos/configdep.py
index 141c7baf7..e0fe1ddac 100644
--- a/python/vyos/configdep.py
+++ b/python/vyos/configdep.py
@@ -33,7 +33,7 @@ if typing.TYPE_CHECKING:
dependency_dir = os.path.join(directories['data'],
'config-mode-dependencies')
-dependent_func: dict[str, list[typing.Callable]] = {}
+dependency_list: list[typing.Callable] = []
DEBUG = False
@@ -122,26 +122,24 @@ def def_closure(target: str, config: 'Config',
def set_dependents(case: str, config: 'Config',
tagnode: typing.Optional[str] = None):
- global dependent_func
+ global dependency_list
+
+ dependency_list = config.dependency_list
d = get_dependency_dict(config)
k = caller_name()
-
- if hasattr(config, 'dependent_func'):
- dependent_func = getattr(config, 'dependent_func')
-
- l = dependent_func.setdefault(k, [])
+ l = dependency_list
for target in d[k][case]:
func = def_closure(target, config, tagnode)
append_uniq(l, func)
- debug_print(f'set_dependents: caller {k}, dependents {names_of(l)}')
+ debug_print(f'set_dependents: caller {k}, current dependents {names_of(l)}')
def call_dependents():
k = caller_name()
- l = dependent_func.get(k, [])
- debug_print(f'call_dependents: caller {k}, dependents {names_of(l)}')
+ l = dependency_list
+ debug_print(f'call_dependents: caller {k}, remaining dependents {names_of(l)}')
while l:
f = l.pop(0)
debug_print(f'calling: {f.__name__}')
diff --git a/src/services/vyos-configd b/src/services/vyos-configd
index cfbf7c03d..87f7c0e25 100755
--- a/src/services/vyos-configd
+++ b/src/services/vyos-configd
@@ -220,6 +220,7 @@ def process_node_data(config, data, last: bool = False) -> int:
script_name = None
args = []
+ config.dependency_list.clear()
res = re.match(r'^(VYOS_TAGNODE_VALUE=[^/]+)?.*\/([^/]+).py(.*)', data)
if res.group(1):