Merge branch 'fog' 60/7460/2
authortierno <alfonso.tiernosepulveda@telefonica.com>
Tue, 14 May 2019 11:22:45 +0000 (11:22 +0000)
committertierno <alfonso.tiernosepulveda@telefonica.com>
Tue, 14 May 2019 11:50:23 +0000 (11:50 +0000)
Change-Id: I22026c17fd94e7a1bb22c0fb87035c4f9bff12ff
Signed-off-by: tierno <alfonso.tiernosepulveda@telefonica.com>
Dockerfile
docker/Dockerfile-local
openmanod
osm_ro/nfvo.py
osm_ro/vim_thread.py
osm_ro/vimconn_openstack.py
osm_ro/vimconn_vmware.py
scripts/install-openmano.sh

index 10efe45..ebbd0ac 100644 (file)
@@ -33,6 +33,7 @@ RUN  apt-get update && \
   DEBIAN_FRONTEND=noninteractive apt-get update && \
   DEBIAN_FRONTEND=noninteractive apt-get -y install python-novaclient python-keystoneclient python-glanceclient python-cinderclient python-neutronclient python-networking-l2gw && \
   DEBIAN_FRONTEND=noninteractive pip install -U progressbar pyvmomi pyvcloud==19.1.1 && \
+  DEBIAN_FRONTEND=noninteractive pip install -U fog05rest && \
   DEBIAN_FRONTEND=noninteractive apt-get -y install python-argcomplete python-bottle python-cffi python-packaging python-paramiko python-pkgconfig libmysqlclient-dev libssl-dev libffi-dev python-mysqldb && \
   DEBIAN_FRONTEND=noninteractive apt-get -y install python-logutils python-openstackclient python-openstacksdk && \
   DEBIAN_FRONTEND=noninteractive pip install untangle && \
index ec9a0ed..1bd50a4 100644 (file)
@@ -10,6 +10,8 @@ RUN apt-get update && \
     DEBIAN_FRONTEND=noninteractive apt-get -y install wget tox && \
     DEBIAN_FRONTEND=noninteractive pip2 install pip==9.0.3 && \
     DEBIAN_FRONTEND=noninteractive pip2 install -U progressbar pyvmomi pyvcloud==19.1.1 && \
+    DEBIAN_FRONTEND=noninteractive pip2 install -U fog05rest && \
+    DEBIAN_FRONTEND=noninteractive apt-get -y install python-requests && \
     DEBIAN_FRONTEND=noninteractive apt-get -y install python-novaclient python-keystoneclient python-glanceclient python-cinderclient python-neutronclient python-networking-l2gw && \
     DEBIAN_FRONTEND=noninteractive apt-get -y install python-cffi libmysqlclient-dev libssl-dev libffi-dev python-mysqldb && \
     DEBIAN_FRONTEND=noninteractive apt-get -y install python-openstacksdk python-openstackclient && \
@@ -21,14 +23,7 @@ RUN apt-get update && \
 
 COPY . /root/RO
 
-RUN apt-get -y install python-requests && \
-    mkdir -p /root/fos && \
-    cd  /root/fos && \
-    git clone https://github.com/eclipse/fog05 && \
-    cd fog05/src/api/python/rest_api/ && \
-    python setup.py install && \
-    cd ~ && \
-    /root/RO/scripts/install-osm-im.sh --develop && \
+RUN /root/RO/scripts/install-osm-im.sh --develop && \
     /root/RO/scripts/install-lib-osm-openvim.sh --develop && \
     make -C /root/RO prepare && \
     mkdir -p /var/log/osm && \
index 398d685..d16c8dd 100755 (executable)
--- a/openmanod
+++ b/openmanod
@@ -41,6 +41,9 @@ from jsonschema import validate as js_v, exceptions as js_e
 import logging
 import logging.handlers as log_handlers
 import socket
+
+from yaml import MarkedYAMLError
+
 from osm_ro import httpserver, nfvo, nfvo_db
 from osm_ro.openmano_schemas import config_schema
 from osm_ro.db_base import db_base_Exception
@@ -72,33 +75,33 @@ def load_configuration(configuration_file):
                       'auto_push_VNF_to_VIMs': True,
                       'db_host': 'localhost',
                       'db_ovim_host': 'localhost'
-    }
+                      }
     try:
-        #Check config file exists
+        # Check config file exists
         with open(configuration_file, 'r') as f:
             config_str = f.read()
-        #Parse configuration file
+        # Parse configuration file
         config = yaml.load(config_str)
-        #Validate configuration file with the config_schema
+        # Validate configuration file with the config_schema
         js_v(config, config_schema)
 
-        #Add default values tokens
-        for k,v in default_tokens.items():
+        # Add default values tokens
+        for k, v in default_tokens.items():
             if k not in config:
-                config[k]=v
+                config[k] = v
         return config
 
     except yaml.YAMLError as e:
         error_pos = ""
-        if hasattr(e, 'problem_mark'):
+        if isinstance(e, MarkedYAMLError):
             mark = e.problem_mark
-            error_pos = " at line:{} column:{}".format(mark.line+1, mark.column+1)
+            error_pos = " at line:{} column:{}".format(mark.line + 1, mark.column + 1)
         raise LoadConfigurationException("Bad YAML format at configuration file '{file}'{pos}: {message}".format(
             file=configuration_file, pos=error_pos, message=e))
     except js_e.ValidationError as e:
         error_pos = ""
         if e.path:
-            error_pos=" at '" + ":".join(map(str, e.path))+"'"
+            error_pos = " at '" + ":".join(map(str, e.path)) + "'"
         raise LoadConfigurationException("Invalid field at configuration file '{file}'{pos} {message}".format(
             file=configuration_file, pos=error_pos, message=e))
     except Exception as e:
@@ -107,20 +110,18 @@ def load_configuration(configuration_file):
 
 
 def console_port_iterator():
-    '''this iterator deals with the http_console_ports
+    """
+    this iterator deals with the http_console_ports
     returning the ports one by one
-    '''
+    """
     index = 0
     while index < len(global_config["http_console_ports"]):
         port = global_config["http_console_ports"][index]
-        #print("ports -> ", port)
         if type(port) is int:
             yield port
-        else: #this is dictionary with from to keys
+        else:  # this is dictionary with from to keys
             port2 = port["from"]
-            #print("ports -> ", port, port2)
             while port2 <= port["to"]:
-                #print("ports -> ", port, port2)
                 yield port2
                 port2 += 1
         index += 1
@@ -131,13 +132,15 @@ def usage():
     print("      -v|--version: prints current version")
     print("      -c|--config [configuration_file]: loads the configuration file (default: openmanod.cfg)")
     print("      -h|--help: shows this help")
-    print("      -p|--port [port_number]: changes port number and overrides the port number in the configuration file (default: 9090)")
-    print("      -P|--adminport [port_number]: changes admin port number and overrides the port number in the configuration file (default: 9095)")
-    # print( "      -V|--vnf-repository: changes the path of the vnf-repository and overrides the path in the configuration file")
+    print(
+        "      -p|--port [port_number]: changes port number and overrides the port number in the configuration file (default: 9090)")
+    print(
+        "      -P|--adminport [port_number]: changes admin port number and overrides the port number in the configuration file (default: 9095)")
     print("      --log-socket-host HOST: send logs to this host")
     print("      --log-socket-port PORT: send logs using this port (default: 9022)")
     print("      --log-file FILE: send logs to this file")
-    print("      --create-tenant NAME: Try to creates this tenant name before starting, ignoring any errors as e.g. conflict")
+    print(
+        "      --create-tenant NAME: Try to creates this tenant name before starting, ignoring any errors as e.g. conflict")
     return
 
 
@@ -146,7 +149,6 @@ def set_logging_file(log_file):
         file_handler = logging.handlers.RotatingFileHandler(log_file, maxBytes=100e6, backupCount=9, delay=0)
         file_handler.setFormatter(log_formatter_simple)
         logger.addHandler(file_handler)
-        # logger.debug("moving logs to '%s'", global_config["log_file"])
         # remove initial stream handler
         logging.root.removeHandler(logging.root.handlers[0])
         print ("logging on '{}'".format(log_file))
@@ -155,34 +157,28 @@ def set_logging_file(log_file):
             "Cannot open logging file '{}': {}. Check folder exist and permissions".format(log_file, e))
 
 
-if __name__=="__main__":
-    # env2config contains envioron variable names and the correspondence with configuration file openmanod.cfg keys.
+if __name__ == "__main__":
+    # env2config contains environ variable names and the correspondence with configuration file openmanod.cfg keys.
     # If this environ is defined, this value is taken instead of the one at at configuration file
     env2config = {
         'RO_DB_HOST': 'db_host',
         'RO_DB_NAME': 'db_name',
         'RO_DB_USER': 'db_user',
         'RO_DB_PASSWORD': 'db_passwd',
-        # 'RO_DB_PORT': 'db_port',
         'RO_DB_OVIM_HOST': 'db_ovim_host',
         'RO_DB_OVIM_NAME': 'db_ovim_name',
         'RO_DB_OVIM_USER': 'db_ovim_user',
         'RO_DB_OVIM_PASSWORD': 'db_ovim_passwd',
-        # 'RO_DB_OVIM_PORT': 'db_ovim_port',
         'RO_LOG_LEVEL': 'log_level',
         'RO_LOG_FILE': 'log_file',
     }
     # Configure logging step 1
     hostname = socket.gethostname()
-    # streamformat = "%(levelname)s (%(module)s:%(lineno)d) %(message)s"
-    # "%(asctime)s %(name)s %(levelname)s %(filename)s:%(lineno)d %(funcName)s %(process)d: %(message)s"
-    log_formatter_complete = logging.Formatter('%(asctime)s.%(msecs)03d00Z[{host}@openmanod] %(filename)s:%(lineno)s '
-                                               'severity:%(levelname)s logger:%(name)s log:%(message)s'.format(
-                                                    host=hostname),
-                                               datefmt='%Y-%m-%dT%H:%M:%S')
-    log_format_simple =  "%(asctime)s %(levelname)s  %(name)s %(thread)d %(filename)s:%(lineno)s %(message)s"
+    log_formatter_str = '%(asctime)s.%(msecs)03d00Z[{host}@openmanod] %(filename)s:%(lineno)s severity:%(levelname)s logger:%(name)s log:%(message)s'
+    log_formatter_complete = logging.Formatter(log_formatter_str.format(host=hostname), datefmt='%Y-%m-%dT%H:%M:%S')
+    log_format_simple = "%(asctime)s %(levelname)s  %(name)s %(thread)d %(filename)s:%(lineno)s %(message)s"
     log_formatter_simple = logging.Formatter(log_format_simple, datefmt='%Y-%m-%dT%H:%M:%S')
-    logging.basicConfig(format=log_format_simple, level= logging.DEBUG)
+    logging.basicConfig(format=log_format_simple, level=logging.DEBUG)
     logger = logging.getLogger('openmano')
     logger.setLevel(logging.DEBUG)
     socket_handler = None
@@ -193,7 +189,7 @@ if __name__=="__main__":
         opts, args = getopt.getopt(sys.argv[1:], "hvc:V:p:P:",
                                    ["config=", "help", "version", "port=", "vnf-repository=", "adminport=",
                                     "log-socket-host=", "log-socket-port=", "log-file=", "create-tenant="])
-        port=None
+        port = None
         port_admin = None
         config_file = 'osm_ro/openmanod.cfg'
         vnf_repository = None
@@ -233,7 +229,6 @@ if __name__=="__main__":
         global_config = load_configuration(config_file)
         global_config["version"] = __version__
         global_config["version_date"] = version_date
-        #print global_config
         # Override parameters obtained by command line on ENV
         if port:
             global_config['http_port'] = port
@@ -250,52 +245,37 @@ if __name__=="__main__":
                 if not env_k.startswith("RO_") or env_k not in env2config or not env_v:
                     continue
                 global_config[env2config[env_k]] = env_v
-                if env_k.endswith("PORT"):    # convert to int, skip if not possible
+                if env_k.endswith("PORT"):  # convert to int, skip if not possible
                     global_config[env2config[env_k]] = int(env_v)
             except Exception as e:
                 logger.warn("skipping environ '{}={}' because exception '{}'".format(env_k, env_v, e))
 
-#         if vnf_repository is not None:
-#             global_config['vnf_repository'] = vnf_repository
-#         else:
-#             if not 'vnf_repository' in global_config:
-#                 logger.error( os.getcwd() )
-#                 global_config['vnf_repository'] = os.getcwd()+'/vnfrepo'
-#         #print global_config
-#         if not os.path.exists(global_config['vnf_repository']):
-#             logger.error( "Creating folder vnf_repository folder: '%s'.", global_config['vnf_repository'])
-#             try:
-#                 os.makedirs(global_config['vnf_repository'])
-#             except Exception as e:
-#                 logger.error( "Error '%s'. Ensure the path 'vnf_repository' is properly set at %s",e.args[1], config_file)
-#                 exit(-1)
-
         global_config["console_port_iterator"] = console_port_iterator
-        global_config["console_thread"]={}
-        global_config["console_ports"]={}
+        global_config["console_thread"] = {}
+        global_config["console_ports"] = {}
         if not global_config["http_console_host"]:
             global_config["http_console_host"] = global_config["http_host"]
-            if global_config["http_host"]=="0.0.0.0":
+            if global_config["http_host"] == "0.0.0.0":
                 global_config["http_console_host"] = socket.gethostname()
 
         # Configure logging STEP 2
         if "log_host" in global_config:
-            socket_handler= log_handlers.SocketHandler(global_config["log_socket_host"], global_config["log_socket_port"])
+            socket_handler = log_handlers.SocketHandler(global_config["log_socket_host"],
+                                                        global_config["log_socket_port"])
             socket_handler.setFormatter(log_formatter_complete)
-            if global_config.get("log_socket_level") and global_config["log_socket_level"] != global_config["log_level"]:
+            if global_config.get("log_socket_level") \
+                    and global_config["log_socket_level"] != global_config["log_level"]:
                 socket_handler.setLevel(global_config["log_socket_level"])
             logger.addHandler(socket_handler)
 
-        # logger.addHandler(log_handlers.SysLogHandler())
         if log_file:
             global_config['log_file'] = log_file
         elif global_config.get('log_file'):
             set_logging_file(global_config['log_file'])
 
-        # logging.basicConfig(level = getattr(logging, global_config.get('log_level',"debug")))
         logger.setLevel(getattr(logging, global_config['log_level']))
         logger.critical("Starting openmano server version: '%s %s' command: '%s'",
-                         __version__, version_date, " ".join(sys.argv))
+                        __version__, version_date, " ".join(sys.argv))
 
         for log_module in ("nfvo", "http", "vim", "wim", "db", "console", "ovim"):
             log_level_module = "log_level_" + log_module
@@ -312,14 +292,13 @@ if __name__=="__main__":
                 except IOError as e:
                     raise LoadConfigurationException(
                         "Cannot open logging file '{}': {}. Check folder exist and permissions".format(
-                            global_config[log_file_module], str(e)) )
-            global_config["logger_"+log_module] = logger_module
-        #httpserver.logger = global_config["logger_http"]
-        #nfvo.logger = global_config["logger_nfvo"]
+                            global_config[log_file_module], str(e)))
+            global_config["logger_" + log_module] = logger_module
 
         # Initialize DB connection
-        mydb = nfvo_db.nfvo_db();
-        mydb.connect(global_config['db_host'], global_config['db_user'], global_config['db_passwd'], global_config['db_name'])
+        mydb = nfvo_db.nfvo_db()
+        mydb.connect(global_config['db_host'], global_config['db_user'], global_config['db_passwd'],
+                     global_config['db_name'])
         db_path = osm_ro.__path__[0] + "/database_utils"
         if not os_path.exists(db_path + "/migrate_mano_db.sh"):
             db_path = osm_ro.__path__[0] + "/../database_utils"
@@ -327,22 +306,23 @@ if __name__=="__main__":
             r = mydb.get_db_version()
             if r[0] != database_version:
                 logger.critical("DATABASE wrong version '{current}'. Try to upgrade/downgrade to version '{target}'"
-                                " with '{db_path}/migrate_mano_db.sh {target}'".format(
-                                current=r[0], target=database_version,  db_path=db_path))
+                                " with '{db_path}/migrate_mano_db.sh {target}'".format(current=r[0],
+                                                                                       target=database_version,
+                                                                                       db_path=db_path))
                 exit(-1)
         except db_base_Exception as e:
             logger.critical("DATABASE is not valid. If you think it is corrupted, you can init it with"
                             " '{db_path}/init_mano_db.sh' script".format(db_path=db_path))
             exit(-1)
 
-        nfvo.global_config=global_config
+        nfvo.global_config = global_config
         if create_tenant:
             try:
                 nfvo.new_tenant(mydb, {"name": create_tenant})
             except Exception as e:
                 if isinstance(e, nfvo.NfvoException) and e.http_code == 409:
                     pass  # if tenant exist (NfvoException error 409), ignore
-                else:     # otherwise print and error and continue
+                else:  # otherwise print and error and continue
                     logger.error("Cannot create tenant '{}': {}".format(create_tenant, e))
 
         # WIM module
@@ -359,7 +339,8 @@ if __name__=="__main__":
 
         httpthread.start()
         if 'http_admin_port' in global_config:
-            httpthreadadmin = httpserver.httpserver(mydb, True, global_config['http_host'], global_config['http_admin_port'])
+            httpthreadadmin = httpserver.httpserver(mydb, True, global_config['http_host'],
+                                                    global_config['http_admin_port'])
             httpthreadadmin.start()
         time.sleep(1)
         logger.info('Waiting for http clients')
@@ -369,10 +350,10 @@ if __name__=="__main__":
         time.sleep(20)
         sys.stdout.flush()
 
-        #TODO: Interactive console must be implemented here instead of join or sleep
+        # TODO: Interactive console must be implemented here instead of join or sleep
 
-        #httpthread.join()
-        #if 'http_admin_port' in global_config:
+        # httpthread.join()
+        # if 'http_admin_port' in global_config:
         #    httpthreadadmin.join()
         while True:
             time.sleep(86400)
@@ -382,8 +363,7 @@ if __name__=="__main__":
     except SystemExit:
         pass
     except getopt.GetoptError as e:
-        logger.critical(str(e)) # will print something like "option -a not recognized"
-        #usage()
+        logger.critical(str(e))  # will print something like "option -a not recognized"
         exit(-1)
     except LoadConfigurationException as e:
         logger.critical(str(e))
@@ -397,4 +377,3 @@ if __name__=="__main__":
     nfvo.stop_service()
     if httpthread:
         httpthread.join(1)
-
index d433ac9..414f365 100644 (file)
@@ -126,11 +126,12 @@ def get_non_used_vim_name(datacenter_name, datacenter_id, tenant_name, tenant_id
     if name not in vim_threads["names"]:
         vim_threads["names"].append(name)
         return name
-    name = datacenter_name[:16] + "." + tenant_name[:16]
-    if name not in vim_threads["names"]:
-        vim_threads["names"].append(name)
-        return name
-    name = datacenter_id + "-" + tenant_id
+    if tenant_name:
+        name = datacenter_name[:16] + "." + tenant_name[:16]
+        if name not in vim_threads["names"]:
+            vim_threads["names"].append(name)
+            return name
+    name = datacenter_id
     vim_threads["names"].append(name)
     return name
 
@@ -237,7 +238,7 @@ def start_service(mydb, persistence=None, wim=None):
             except Exception as e:
                 raise NfvoException("Error at VIM  {}; {}: {}".format(vim["type"], type(e).__name__, e),
                                     httperrors.Internal_Server_Error)
-            thread_name = get_non_used_vim_name(vim['datacenter_name'], vim['vim_tenant_id'], vim['vim_tenant_name'],
+            thread_name = get_non_used_vim_name(vim['datacenter_name'], vim['datacenter_id'], vim['vim_tenant_name'],
                                                 vim['vim_tenant_id'])
             new_thread = vim_thread.vim_thread(task_lock, thread_name, vim['datacenter_name'],
                                                vim['datacenter_tenant_id'], db=db, db_lock=db_lock, ovim=ovim)
@@ -880,6 +881,21 @@ def _lookfor_or_create_image(db_image, mydb, descriptor):
         db_image["uuid"] = image_uuid
         return None
 
+def get_resource_allocation_params(quota_descriptor):
+    """
+    read the quota_descriptor from vnfd and fetch the resource allocation properties from the descriptor object
+    :param quota_descriptor: cpu/mem/vif/disk-io quota descriptor
+    :return: quota params for limit, reserve, shares from the descriptor object
+    """
+    quota = {}
+    if quota_descriptor.get("limit"):
+        quota["limit"] = int(quota_descriptor["limit"])
+    if quota_descriptor.get("reserve"):
+        quota["reserve"] = int(quota_descriptor["reserve"])
+    if quota_descriptor.get("shares"):
+        quota["shares"] = int(quota_descriptor["shares"])
+    return quota
+
 def new_vnfd_v3(mydb, tenant_id, vnf_descriptor):
     """
     Parses an OSM IM vnfd_catalog and insert at DB
@@ -1264,6 +1280,15 @@ def new_vnfd_v3(mydb, tenant_id, vnf_descriptor):
                                 numa["cores"] = max(db_flavor["vcpus"], 1)
                             else:
                                 numa["threads"] = max(db_flavor["vcpus"], 1)
+                            epa_vcpu_set = True
+                    if vdu["guest-epa"].get("cpu-quota") and not epa_vcpu_set:
+                        extended["cpu-quota"] = get_resource_allocation_params(vdu["guest-epa"].get("cpu-quota"))
+                    if vdu["guest-epa"].get("mem-quota"):
+                        extended["mem-quota"] = get_resource_allocation_params(vdu["guest-epa"].get("mem-quota"))
+                    if vdu["guest-epa"].get("disk-io-quota"):
+                        extended["disk-io-quota"] = get_resource_allocation_params(vdu["guest-epa"].get("disk-io-quota"))
+                    if vdu["guest-epa"].get("vif-quota"):
+                        extended["vif-quota"] = get_resource_allocation_params(vdu["guest-epa"].get("vif-quota"))
                 if numa:
                     extended["numas"] = [numa]
                 if extended:
index 5d4463f..44c6ea1 100644 (file)
@@ -63,7 +63,7 @@ The task content is (M: stored at memory, D: stored at database):
             vim_status: VIM status of the element. Stored also at database in the instance_XXX
             vim_info:   Detailed information of a vm/net from the VIM. Stored at database in the instance_XXX but not at
                         vim_wim_actions
-    M   depends:    dict with task_index(from depends_on) to vim_id
+    M   depends:    dict with task_index(from depends_on) to dependency task
     M   params:     same as extra[params]
     MD  error_msg:  descriptive text upon an error.Stored also at database instance_XXX
     MD  created_at: task creation time. The task of creation must be the oldest
@@ -534,9 +534,8 @@ class vim_thread(threading.Thread):
                                 task_dependency["instance_action_id"], task_dependency["task_index"],
                                 task_dependency["action"], task_dependency["item"], task_dependency.get("error_msg")))
 
-                    task["depends"]["TASK-"+str(task_index)] = task_dependency["vim_id"]
-                    task["depends"]["TASK-{}.{}".format(task["instance_action_id"], task_index)] =\
-                        task_dependency["vim_id"]
+                    task["depends"]["TASK-"+str(task_index)] = task_dependency
+                    task["depends"]["TASK-{}.{}".format(task["instance_action_id"], task_index)] = task_dependency
                 if dependency_not_completed:
                     # Move this task to the time dependency is going to be modified plus 10 seconds.
                     self.db.update_rows("vim_wim_actions", modified_time=dependency_modified_at + 10,
@@ -810,7 +809,7 @@ class vim_thread(threading.Thread):
             net_list = params[5]
             for net in net_list:
                 if "net_id" in net and is_task_id(net["net_id"]):  # change task_id into network_id
-                    network_id = task["depends"][net["net_id"]]
+                    network_id = task["depends"][net["net_id"]].get("vim_id")
                     if not network_id:
                         raise VimThreadException(
                             "Cannot create VM because depends on a network not created or found: " +
@@ -1057,7 +1056,8 @@ class vim_thread(threading.Thread):
             dep_id = "TASK-" + str(task["extra"]["depends_on"][0])
             task_id = task["instance_action_id"] + "." + str(task["task_index"])
             error_text = ""
-            interfaces = task.get("depends").get(dep_id).get("extra").get("interfaces")
+            interfaces = task["depends"][dep_id]["extra"].get("interfaces")
+
             ingress_interface_id = task.get("extra").get("params").get("ingress_interface_id")
             egress_interface_id = task.get("extra").get("params").get("egress_interface_id")
             ingress_vim_interface_id = None
index 3515ef6..ff67f6f 100644 (file)
@@ -571,6 +571,8 @@ class vimconnector(vimconn.vimconnector):
                         network_dict["provider:segmentation_id"] = self._generate_vlanID()
 
             network_dict["shared"] = shared
+            if self.config.get("disable_network_port_security"):
+                network_dict["port_security_enabled"] = False
             new_net = self.neutron.create_network({'network':network_dict})
             # print new_net
             # create subnetwork, even if there is no profile
@@ -794,8 +796,8 @@ class vimconnector(vimconn.vimconnector):
             flavor_candidate_data = (10000, 10000, 10000)
             flavor_target = (flavor_dict["ram"], flavor_dict["vcpus"], flavor_dict["disk"])
             # numa=None
-            numas = flavor_dict.get("extended", {}).get("numas")
-            if numas:
+            extended = flavor_dict.get("extended", {})
+            if extended:
                 #TODO
                 raise vimconn.vimconnNotFoundException("Flavor with EPA still not implemted")
                 # if len(numas) > 1:
@@ -819,6 +821,20 @@ class vimconnector(vimconn.vimconnector):
         except (nvExceptions.NotFound, nvExceptions.ClientException, ksExceptions.ClientException, ConnectionError) as e:
             self._format_exception(e)
 
+    def process_resource_quota(self, quota, prefix, extra_specs):
+        """
+        :param prefix:
+        :param extra_specs: 
+        :return:
+        """
+        if 'limit' in quota:
+            extra_specs["quota:" + prefix + "_limit"] = quota['limit']
+        if 'reserve' in quota:
+            extra_specs["quota:" + prefix + "_reservation"] = quota['reserve']
+        if 'shares' in quota:
+            extra_specs["quota:" + prefix + "_shares_level"] = "custom"
+            extra_specs["quota:" + prefix + "_shares_share"] = quota['shares']
+
     def new_flavor(self, flavor_data, change_name_if_used=True):
         '''Adds a tenant flavor to openstack VIM
         if change_name_if_used is True, it will change name in case of conflict, because it is not supported name repetition
@@ -846,7 +862,7 @@ class vimconnector(vimconn.vimconnector):
 
                     ram = flavor_data.get('ram',64)
                     vcpus = flavor_data.get('vcpus',1)
-                    numa_properties=None
+                    extra_specs={}
 
                     extended = flavor_data.get("extended")
                     if extended:
@@ -855,13 +871,13 @@ class vimconnector(vimconn.vimconnector):
                             numa_nodes = len(numas)
                             if numa_nodes > 1:
                                 return -1, "Can not add flavor with more than one numa"
-                            numa_properties = {"hw:numa_nodes":str(numa_nodes)}
-                            numa_properties["hw:mem_page_size"] = "large"
-                            numa_properties["hw:cpu_policy"] = "dedicated"
-                            numa_properties["hw:numa_mempolicy"] = "strict"
+                            extra_specs["hw:numa_nodes"] = str(numa_nodes)
+                            extra_specs["hw:mem_page_size"] = "large"
+                            extra_specs["hw:cpu_policy"] = "dedicated"
+                            extra_specs["hw:numa_mempolicy"] = "strict"
                             if self.vim_type == "VIO":
-                                numa_properties["vmware:extra_config"] = '{"numa.nodeAffinity":"0"}'
-                                numa_properties["vmware:latency_sensitivity_level"] = "high"
+                                extra_specs["vmware:extra_config"] = '{"numa.nodeAffinity":"0"}'
+                                extra_specs["vmware:latency_sensitivity_level"] = "high"
                             for numa in numas:
                                 #overwrite ram and vcpus
                                 #check if key 'memory' is present in numa else use ram value at flavor
@@ -871,23 +887,30 @@ class vimconnector(vimconn.vimconnector):
                                 if 'paired-threads' in numa:
                                     vcpus = numa['paired-threads']*2
                                     #cpu_thread_policy "require" implies that the compute node must have an STM architecture
-                                    numa_properties["hw:cpu_thread_policy"] = "require"
-                                    numa_properties["hw:cpu_policy"] = "dedicated"
+                                    extra_specs["hw:cpu_thread_policy"] = "require"
+                                    extra_specs["hw:cpu_policy"] = "dedicated"
                                 elif 'cores' in numa:
                                     vcpus = numa['cores']
                                     # cpu_thread_policy "prefer" implies that the host must not have an SMT architecture, or a non-SMT architecture will be emulated
-                                    numa_properties["hw:cpu_thread_policy"] = "isolate"
-                                    numa_properties["hw:cpu_policy"] = "dedicated"
+                                    extra_specs["hw:cpu_thread_policy"] = "isolate"
+                                    extra_specs["hw:cpu_policy"] = "dedicated"
                                 elif 'threads' in numa:
                                     vcpus = numa['threads']
                                     # cpu_thread_policy "prefer" implies that the host may or may not have an SMT architecture
-                                    numa_properties["hw:cpu_thread_policy"] = "prefer"
-                                    numa_properties["hw:cpu_policy"] = "dedicated"
+                                    extra_specs["hw:cpu_thread_policy"] = "prefer"
+                                    extra_specs["hw:cpu_policy"] = "dedicated"
                                 # for interface in numa.get("interfaces",() ):
                                 #     if interface["dedicated"]=="yes":
                                 #         raise vimconn.vimconnException("Passthrough interfaces are not supported for the openstack connector", http_code=vimconn.HTTP_Service_Unavailable)
                                 #     #TODO, add the key 'pci_passthrough:alias"="<label at config>:<number ifaces>"' when a way to connect it is available
-
+                        elif extended.get("cpu-quota"):
+                            self.process_resource_quota(extended.get("cpu-quota"), "cpu", extra_specs)
+                        if extended.get("mem-quota"):
+                            self.process_resource_quota(extended.get("mem-quota"), "memory", extra_specs)
+                        if extended.get("vif-quota"):
+                            self.process_resource_quota(extended.get("vif-quota"), "vif", extra_specs)
+                        if extended.get("disk-io-quota"):
+                            self.process_resource_quota(extended.get("disk-io-quota"), "disk_io", extra_specs)
                     #create flavor
                     new_flavor=self.nova.flavors.create(name,
                                     ram,
@@ -896,8 +919,8 @@ class vimconnector(vimconn.vimconnector):
                                     is_public=flavor_data.get('is_public', True)
                                 )
                     #add metadata
-                    if numa_properties:
-                        new_flavor.set_keys(numa_properties)
+                    if extra_specs:
+                        new_flavor.set_keys(extra_specs)
                     return new_flavor.id
                 except nvExceptions.Conflict as e:
                     if change_name_if_used and retry < max_retries:
index 94bae39..128e8e1 100644 (file)
@@ -1665,13 +1665,12 @@ class vimconnector(vimconn.vimconnector):
                 else:
                     result = (response.content).replace("\n"," ")
 
-                src = re.search('<Vm goldMaster="false"\sstatus="\d+"\sname="(.*?)"\s'
-                                               'id="(\w+:\w+:vm:.*?)"\shref="(.*?)"\s'
-                              'type="application/vnd\.vmware\.vcloud\.vm\+xml',result)
-                if src:
-                    vm_name = src.group(1)
-                    vm_id = src.group(2)
-                    vm_href = src.group(3)
+                vapp_template_tree = XmlElementTree.fromstring(response.content)
+                children_element = [child for child in vapp_template_tree if 'Children' in child.tag][0]
+                vm_element = [child for child in children_element if 'Vm' in child.tag][0]
+                vm_name = vm_element.get('name')
+                vm_id = vm_element.get('id')
+                vm_href = vm_element.get('href')
 
                 cpus = re.search('<rasd:Description>Number of Virtual CPUs</.*?>(\d+)</rasd:VirtualQuantity>',result).group(1)
                 memory_mb = re.search('<rasd:Description>Memory Size</.*?>(\d+)</rasd:VirtualQuantity>',result).group(1)
index bbf33e4..a34a0aa 100755 (executable)
@@ -262,6 +262,9 @@ then
     [ "$_DISTRO" == "Ubuntu" ] && install_packages "genisoimage"
     [ "$_DISTRO" == "CentOS" -o "$_DISTRO" == "Red" ] && install_packages "genisoimage"
 
+    # required for fog connector
+    pip2 install fog05rest || exit 1
+
     # required for OpenNebula connector
     pip2 install untangle || exit 1
     pip2 install -e git+https://github.com/python-oca/python-oca#egg=oca || exit 1