X-Git-Url: https://osm.etsi.org/gitweb/?a=blobdiff_plain;f=descriptor-packages%2Ftools%2Fvalidate_descriptor.py;h=3c6dd4198dac4a0fcfa384d0fd69e36e40a13945;hb=c33cdc3359627245f605fbc1ab0084e03f30837c;hp=010ba16ff00febd5b206ada7a9c741b9087bcd43;hpb=8bc90dcf25e7daefc206b963ecc1d8cd4d822121;p=osm%2Fdevops.git diff --git a/descriptor-packages/tools/validate_descriptor.py b/descriptor-packages/tools/validate_descriptor.py index 010ba16f..3c6dd419 100755 --- a/descriptor-packages/tools/validate_descriptor.py +++ b/descriptor-packages/tools/validate_descriptor.py @@ -22,6 +22,7 @@ import json import yaml import sys import getopt +import os """ Tests the format of OSM VNFD and NSD descriptors @@ -38,14 +39,16 @@ class ArgumentParserError(Exception): def usage(): print("Usage: {} [options] FILE".format(sys.argv[0])) - print(" EXPERIMENTAL: Validates vnfd, nsd descriptors format") - print(" FILE: a yaml or json vnfd-catalog or nsd-catalog descriptor") + print(" Validates vnfd, nsd and nst descriptors format") + print(" FILE: a yaml or json vnfd-catalog, nsd-catalog or nst descriptor") print(" OPTIONS:") print(" -v|--version: prints current version") print(" -h|--help: shows this help") print(" -i|--input FILE: (same as param FILE) descriptor file to be upgraded") + print(" -c|--charms: looks for the charms folder and validates its coherency with the descriptor") return + def remove_prefix(desc, prefix): """ Recursively removes prefix from keys @@ -68,12 +71,39 @@ def remove_prefix(desc, prefix): if isinstance(desc, (list, tuple, dict)): remove_prefix(i, prefix) -if __name__=="__main__": + +# Mrityunjay Yadav: Function to verify charm included in VNF Package +def validate_charm(charm, desc_file): + """ + Verify charm included in VNF Package and raised error if invalid + :param charm: vnf-configuration/vdu-configuration + :param desc_file: descriptor file + :return: None + """ + check_list = ['layer.yaml', 'metadata.yaml', 'actions.yaml', 'actions', 'hooks'] + charm_name = charm['juju']['charm'] + charm_dir = os.path.join(os.path.abspath(os.path.dirname(desc_file)), 'charms', charm_name) + + config_primitive = charm.get('config-primitive', []) + initial_config_primitive = charm.get('initial-config-primitive', []) + + if charm.get('metrics'): + check_list.append('metrics.yaml') + + if os.path.exists(charm_dir): + if not all(item in os.listdir(charm_dir) for item in check_list): + raise KeyError("Invalid charm {}".format(charm_name)) + else: + raise KeyError("Provided charm:{} does not exist in descriptor.".format(charm_name)) + + +if __name__ == "__main__": error_position = [] format_output_yaml = True input_file_name = None test_file = None file_name = None + validate_charms = False try: # load parameters and configuration opts, args = getopt.getopt(sys.argv[1:], "hvi:o:", ["input=", "help", "version",]) @@ -87,6 +117,8 @@ if __name__=="__main__": sys.exit() elif o in ("-i", "--input"): input_file_name = a + elif o in ("-c", "--charms"): + validate_charms = True else: assert False, "Unhandled option" if not input_file_name: @@ -109,6 +141,7 @@ if __name__=="__main__": import osm_im.vnfd as vnfd_catalog import osm_im.nsd as nsd_catalog + import osm_im.nst as nst_catalog from pyangbind.lib.serialise import pybindJSONDecoder if "vnfd:vnfd-catalog" in data or "vnfd-catalog" in data: @@ -126,24 +159,35 @@ if __name__=="__main__": for external_interface in external_interface_list: if external_interface.get("virtual-interface", {}).get("type") == "OM-MGMT": raise KeyError( - "Wrong 'Virtual-interface type': Deprecated 'OM-MGMT' value. Please, use 'VIRTIO' instead") - interface_list = vdu.pop("interface", ()) + "Wrong 'Virtual-interface type': Deprecated 'OM-MGMT' value. Please, use 'PARAVIRT' instead") + interface_list = vdu.get("interface", ()) for interface in interface_list: if interface.get("virtual-interface", {}).get("type") == "OM-MGMT": raise KeyError( - "Wrong 'Virtual-interface type': Deprecated 'OM-MGMT' value. Please, use 'VIRTIO' instead") + "Wrong 'Virtual-interface type': Deprecated 'OM-MGMT' value. Please, use 'PARAVIRT' instead") + # Mrityunjay yadav: Verify charm if included in vdu + if vdu.get("vdu-configuration", False) and validate_charms: + validate_charm(vdu["vdu-configuration"], input_file_name) if vnfd.get("mgmt-interface"): mgmt_iface = True if vnfd["mgmt-interface"].get("vdu-id"): raise KeyError("'mgmt-iface': Deprecated 'vdu-id' field. Please, use 'cp' field instead") + # Mrityunjay yadav: Verify charm if included in vnf + if vnfd.get("vnf-configuration", False) and validate_charms: + validate_charm(vnfd["vnf-configuration"], input_file_name) + if not mgmt_iface: - raise KeyError("'mgmt-iface' is a mandatory field and it is not defined") + raise KeyError("'mgmt-interface' is a mandatory field and it is not defined") myvnfd = vnfd_catalog.vnfd() pybindJSONDecoder.load_ietf_json(data, None, None, obj=myvnfd) elif "nsd:nsd-catalog" in data or "nsd-catalog" in data: descriptor = "NS" mynsd = nsd_catalog.nsd() pybindJSONDecoder.load_ietf_json(data, None, None, obj=mynsd) + elif "nst:nst" in data or "nst" in data: + descriptor = "NST" + mynst = nst_catalog.nst() + pybindJSONDecoder.load_ietf_json(data, None, None, obj=mynst) else: descriptor = None raise KeyError("This is not neither nsd-catalog nor vnfd-catalog descriptor")