diff options
author | Nataliia Solomko <natalirs1985@gmail.com> | 2024-06-06 13:55:25 +0300 |
---|---|---|
committer | Nataliia Solomko <natalirs1985@gmail.com> | 2024-06-06 18:39:50 +0300 |
commit | 61f8250184e927de9ab6bddc207b917bef7da42b (patch) | |
tree | 9ac7eb51bf728a3fa5b5ce07d95d162e9a287917 /scripts/build-command-templates | |
parent | ea477ed0cec87041cb94c0eb2bc2ce9523a92a0a (diff) | |
download | vyos-1x-61f8250184e927de9ab6bddc207b917bef7da42b.tar.gz vyos-1x-61f8250184e927de9ab6bddc207b917bef7da42b.zip |
xml: T6423: enforce priority on nodes having an owner
Diffstat (limited to 'scripts/build-command-templates')
-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) |