summaryrefslogtreecommitdiff
path: root/scripts
diff options
context:
space:
mode:
authorNataliia Solomko <natalirs1985@gmail.com>2024-06-06 13:55:25 +0300
committerChristian Breunig <christian@breunig.cc>2024-06-09 08:49:23 +0200
commit6ec4be553be88da4540554c6f36ab051c565dfcf (patch)
tree800ec533756a4d39a1e7bc4f2da0372f373c7ff3 /scripts
parent07efcfc28cc2ba1420a470f9bb9cf3be68d8ff47 (diff)
downloadvyos-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-xscripts/build-command-templates12
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)