Fix vnflcm adaptor pep8 error.
[vfc/gvnfm/vnflcm.git] / lcm / lcm / pub / vimapi / adaptor.py
index 9cbc7b1..e0d8cb7 100644 (file)
@@ -13,9 +13,7 @@
 # limitations under the License.
 
 import logging
-import sys
 import time
-import traceback
 
 from lcm.pub.utils.values import ignore_case_get, set_opt_val
 from . import api
@@ -47,6 +45,7 @@ def get_tenant_id(vim_cache, vim_id, tenant_name):
         raise VimException("Tenant(%s) not found in vim(%s)" % (tenant_name, vim_id), ERR_CODE)
     return vim_cache[vim_id][tenant_name]
 
+
 def set_res_cache(res_cache, res_type, key, val):
     if res_type not in res_cache:
         res_cache[res_type] = {}
@@ -54,6 +53,7 @@ def set_res_cache(res_cache, res_type, key, val):
         raise VimException("Duplicate key(%s) of %s" % (key, res_type), ERR_CODE)
     res_cache[res_type][key] = val
 
+
 def get_res_id(res_cache, res_type, key):
     if res_type not in res_cache:
         raise VimException("%s not found in cache" % res_type, ERR_CODE)
@@ -61,6 +61,7 @@ def get_res_id(res_cache, res_type, key):
         raise VimException("%s(%s) not found in cache" % (res_type, key), ERR_CODE)
     return res_cache[res_type][key]
 
+
 def create_vim_res(data, do_notify):
     vim_cache, res_cache = {}, {}
     for vol in ignore_case_get(data, "volume_storages"):
@@ -76,19 +77,22 @@ def create_vim_res(data, do_notify):
     for vm in ignore_case_get(data, "vdus"):
         create_vm(vim_cache, res_cache, data, vm, do_notify, RES_VM)
 
+
 def delete_vim_res(data, do_notify):
     res_types = [RES_VM, RES_FLAVOR, RES_PORT, RES_SUBNET, RES_NETWORK, RES_VOLUME]
-    res_del_funs = [api.delete_vm, api.delete_flavor, api.delete_port, 
-        api.delete_subnet, api.delete_network, api.delete_volume]
+    res_del_funs = [api.delete_vm, api.delete_flavor, api.delete_port,
+                    api.delete_subnet, api.delete_network, api.delete_volume]
     for res_type, res_del_fun in zip(res_types, res_del_funs):
         for res in ignore_case_get(data, res_type):
             try:
-                res_del_fun(res["vim_id"], res["tenant_id"], res["res_id"])
+                if 1 == res["is_predefined"]:
+                    res_del_fun(res["vim_id"], res["tenant_id"], res["res_id"])
             except VimException as e:
                 logger.error("Failed to delete %s(%s)", res_type, res["res_id"])
                 logger.error("%s:%s", e.http_code, e.message)
             do_notify(res_type, res["res_id"])
 
+
 def create_volume(vim_cache, res_cache, vol, do_notify, res_type):
     location_info = vol["properties"]["location_info"]
     param = {
@@ -103,7 +107,7 @@ def create_volume(vim_cache, res_cache, vol, do_notify, res_type):
     ret = api.create_volume(vim_id, tenant_id, param)
     ret["nodeId"] = vol["volume_storage_id"]
     do_notify(res_type, ret)
-    vol_id, vol_name, return_code = ret["id"], ret["name"], ret["returnCode"]
+    vol_id, vol_name = ret["id"], ret["name"]
     set_res_cache(res_cache, res_type, vol["volume_storage_id"], vol_id)
     retry_count, max_retry_count = 0, 300
     while retry_count < max_retry_count:
@@ -114,7 +118,8 @@ def create_volume(vim_cache, res_cache, vol, do_notify, res_type):
         time.sleep(2)
         retry_count = retry_count + 1
     raise VimException("Failed to create Volume(%s): Timeout." % vol_name, ERR_CODE)
-    
+
+
 def create_network(vim_cache, res_cache, network, do_notify, res_type):
     location_info = network["properties"]["location_info"]
     param = {
@@ -132,7 +137,8 @@ def create_network(vim_cache, res_cache, network, do_notify, res_type):
     ret["nodeId"] = network["vl_id"]
     do_notify(res_type, ret)
     set_res_cache(res_cache, res_type, network["vl_id"], ret["id"])
-    
+
+
 def create_subnet(vim_cache, res_cache, subnet, do_notify, res_type):
     location_info = subnet["properties"]["location_info"]
     network_id = get_res_id(res_cache, RES_NETWORK, subnet["vl_id"])
@@ -156,13 +162,16 @@ def create_subnet(vim_cache, res_cache, subnet, do_notify, res_type):
     ret = api.create_subnet(vim_id, tenant_id, param)
     do_notify(res_type, ret)
     set_res_cache(res_cache, res_type, subnet["vl_id"], ret["id"])
-    
+
+
 def create_port(vim_cache, res_cache, data, port, do_notify, res_type):
     location_info = None
     port_ref_vdu_id = ignore_case_get(port, "vdu_id")
     for vdu in ignore_case_get(data, "vdus"):
         if vdu["vdu_id"] == port_ref_vdu_id:
             location_info = vdu["properties"]["location_info"]
+            if port["cp_id"] not in vdu["cps"]:
+                vdu["cps"].append(port["cp_id"])
             break
     if not location_info:
         err_msg = "vdu_id(%s) for cp(%s) is not defined"
@@ -174,13 +183,13 @@ def create_port(vim_cache, res_cache, data, port, do_notify, res_type):
         subnet_id = get_res_id(res_cache, RES_SUBNET, port["vl_id"])
     param = {
         "networkId": network_id,
-        "name": port["properties"]["name"]
+        "name": port["properties"].get("name", "undefined")
     }
     set_opt_val(param, "subnetId", subnet_id)
     set_opt_val(param, "macAddress", ignore_case_get(port["properties"], "mac_address"))
     set_opt_val(param, "ip", ignore_case_get(port["properties"], "ip_address"))
     set_opt_val(param, "vnicType", ignore_case_get(port["properties"], "vnic_type"))
-    set_opt_val(param, "securityGroups", "") # TODO
+    set_opt_val(param, "securityGroups", "")   # TODO
     vim_id, tenant_name = location_info["vimid"], location_info["tenant"]
     tenant_id = get_tenant_id(vim_cache, vim_id, tenant_name)
     ret = api.create_port(vim_id, tenant_id, param)
@@ -188,6 +197,7 @@ def create_port(vim_cache, res_cache, data, port, do_notify, res_type):
     do_notify(res_type, ret)
     set_res_cache(res_cache, res_type, port["cp_id"], ret["id"])
 
+
 def create_flavor(vim_cache, res_cache, data, flavor, do_notify, res_type):
     location_info = flavor["properties"]["location_info"]
     local_storages = ignore_case_get(data, "local_storages")
@@ -202,7 +212,7 @@ def create_flavor(vim_cache, res_cache, data, flavor, do_notify, res_type):
             if local_storage_id != local_storage["local_storage_id"]:
                 continue
             disk_type = local_storage["properties"]["disk_type"]
-            disk_size = int(local_storage["properties"]["size"].replace('GB', '').strip())*1024
+            disk_size = int(local_storage["properties"]["size"].replace('GB', '').strip())
             if disk_type == "root":
                 param["disk"] = disk_size
             elif disk_type == "ephemeral":
@@ -219,13 +229,14 @@ def create_flavor(vim_cache, res_cache, data, flavor, do_notify, res_type):
     ret = api.create_flavor(vim_id, tenant_id, param)
     do_notify(res_type, ret)
     set_res_cache(res_cache, res_type, flavor["vdu_id"], ret["id"])
-    
+
+
 def create_vm(vim_cache, res_cache, data, vm, do_notify, res_type):
     location_info = vm["properties"]["location_info"]
     vim_id, tenant_name = location_info["vimid"], location_info["tenant"]
     tenant_id = get_tenant_id(vim_cache, vim_id, tenant_name)
     param = {
-        "name": vm["properties"]["name"],
+        "name": vm["properties"].get("name", "undefined"),
         "flavorId": get_res_id(res_cache, RES_FLAVOR, vm["vdu_id"]),
         "boot": {},
         "nicArray": [],
@@ -238,8 +249,8 @@ def create_vm(vim_cache, res_cache, data, vm, do_notify, res_type):
         img_name = ""
         for img in ignore_case_get(data, "image_files"):
             if vm["image_file"] == img["image_file_id"]:
-               img_name = img["properties"]["name"]
-               break
+                img_name = img["properties"]["name"]
+                break
         if not img_name:
             raise VimException("Undefined image(%s)" % vm["image_file"], ERR_CODE)
         images = api.list_image(vim_id, tenant_id)
@@ -272,14 +283,17 @@ def create_vm(vim_cache, res_cache, data, vm, do_notify, res_type):
         })
 
     set_opt_val(param, "availabilityZone", ignore_case_get(location_info, "availability_zone"))
-    set_opt_val(param, "userdata", "") # TODO Configuration information or scripts to use upon launch
-    set_opt_val(param, "metadata", "") # TODO [{"keyName": "foo", "value": "foo value"}]
-    set_opt_val(param, "securityGroups", "") # TODO List of names of security group
-    set_opt_val(param, "serverGroup", "") # TODO the ServerGroup for anti-affinity and affinity
-    
+    set_opt_val(param, "userdata", "")         # TODO Configuration information or scripts to use upon launch
+    set_opt_val(param, "metadata", "")         # TODO [{"keyName": "foo", "value": "foo value"}]
+    set_opt_val(param, "securityGroups", "")   # TODO List of names of security group
+    set_opt_val(param, "serverGroup", "")      # TODO the ServerGroup for anti-affinity and affinity
+
     ret = api.create_vm(vim_id, tenant_id, param)
     do_notify(res_type, ret)
-    vm_id, vm_name, return_code = ret["id"], ret["name"], ret["returnCode"]
+    vm_id = ret["id"]
+    if ignore_case_get(ret, "name"):
+        vm_name = vm["properties"].get("name", "undefined")
+        logger.debug("vm_name:%s" % vm_name)
     opt_vm_status = "Timeout"
     retry_count, max_retry_count = 0, 100
     while retry_count < max_retry_count: