diff options
author | Nataliia Solomko <natalirs1985@gmail.com> | 2024-06-06 13:55:25 +0300 |
---|---|---|
committer | Christian Breunig <christian@breunig.cc> | 2024-06-09 08:49:23 +0200 |
commit | 6ec4be553be88da4540554c6f36ab051c565dfcf (patch) | |
tree | 800ec533756a4d39a1e7bc4f2da0372f373c7ff3 /scripts | |
parent | 07efcfc28cc2ba1420a470f9bb9cf3be68d8ff47 (diff) | |
download | vyos-1x-6ec4be553be88da4540554c6f36ab051c565dfcf.tar.gz vyos-1x-6ec4be553be88da4540554c6f36ab051c565dfcf.zip |
xml: T6423: enforce priority on nodes having an owner
(cherry picked from commit 61f8250184e927de9ab6bddc207b917bef7da42b)
Diffstat (limited to 'scripts')
-rwxr-xr-x | scripts/build-command-templates | 12 |
1 files changed, 11 insertions, 1 deletions
diff --git a/scripts/build-command-templates b/scripts/build-command-templates index 2e7f8b994..36929abb2 100755 --- a/scripts/build-command-templates +++ b/scripts/build-command-templates @@ -287,6 +287,12 @@ def process_node(n, tmpl_dir): props = get_properties(props_elem, n.find("defaultValue")) if owner: props["owner"] = owner + # <priority> tag is mandatory if the parent node has an owner + if "priority" not in props: + raise ValueError( + f"<priority> tag should be set for the node <{name}> path '{' '.join(my_tmpl_dir[1:])}'" + ) + # Type should not be set for non-tag, non-leaf nodes # For non-valueless leaf nodes, set the type to txt: to make them have some type, # actual value validation is handled by constraints translated to syntax:expression: @@ -335,4 +341,8 @@ nodes = root.iterfind("*") for n in nodes: if n.tag == "syntaxVersion": continue - process_node(n, [output_dir]) + try: + process_node(n, [output_dir]) + except ValueError as e: + print(e) + sys.exit(1) |