aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorfujinhua <fu.jinhua@zte.com.cn>2018-10-18 15:42:20 +0800
committerfujinhua <fu.jinhua@zte.com.cn>2018-10-18 15:42:20 +0800
commitb422b4f7d22e815077a518ab5ba3aa42694f9e2c (patch)
tree6cc4c13e345c24e98e40dfe44dc22e188fb2f2a6
parent6979eccfb75623febf332c237d9b03d99da48e80 (diff)
Add metadata id check for vnf/pnf pkg
Change-Id: I49f4b7c4f339d944e4d1ee9f8df154e95cb5a0ad Issue-ID: VFC-1151 Signed-off-by: fujinhua <fu.jinhua@zte.com.cn>
-rw-r--r--catalog/packages/biz/pnf_descriptor.py4
-rw-r--r--catalog/packages/biz/vnf_package.py4
2 files changed, 6 insertions, 2 deletions
diff --git a/catalog/packages/biz/pnf_descriptor.py b/catalog/packages/biz/pnf_descriptor.py
index 528fd178..f1dc55b2 100644
--- a/catalog/packages/biz/pnf_descriptor.py
+++ b/catalog/packages/biz/pnf_descriptor.py
@@ -130,7 +130,9 @@ class PnfDescriptor(object):
pnfd_json = toscaparser.parse_pnfd(local_file_name)
pnfd = json.JSONDecoder().decode(pnfd_json)
- pnfd_id = pnfd["metadata"]["id"]
+ pnfd_id = pnfd["metadata"].get("id", '')
+ if not pnfd_id:
+ raise CatalogException("PNFDID(metadata.id) of PNF(%s) does not exist." % pnfd_info_id)
if pnfd_id and PnfPackageModel.objects.filter(pnfdId=pnfd_id):
logger.info('PNFD(%s) already exists.' % pnfd_id)
raise CatalogException("PNFD(%s) already exists." % pnfd_id)
diff --git a/catalog/packages/biz/vnf_package.py b/catalog/packages/biz/vnf_package.py
index be8edba4..53f7fc0a 100644
--- a/catalog/packages/biz/vnf_package.py
+++ b/catalog/packages/biz/vnf_package.py
@@ -194,7 +194,9 @@ def parse_vnfd_and_save(vnf_pkg_id, vnf_pkg_path):
vnfd_json = toscaparser.parse_vnfd(vnf_pkg_path)
vnfd = json.JSONDecoder().decode(vnfd_json)
- vnfd_id = vnfd["metadata"]["id"]
+ vnfd_id = vnfd["metadata"].get("id", '')
+ if not vnfd_id:
+ raise CatalogException("VNFDID(metadata.id) of VNF(%s) does not exist." % vnf_pkg_id)
if VnfPackageModel.objects.filter(vnfdId=vnfd_id):
logger.error("VNF package(%s) already exists." % vnfd_id)
raise CatalogException("VNF package(%s) already exists." % vnfd_id)