Merge 'master' into v7.0 for v7.1.0 79/8879/2
authortierno <alfonso.tiernosepulveda@telefonica.com>
Thu, 7 May 2020 14:15:22 +0000 (14:15 +0000)
committertierno <alfonso.tiernosepulveda@telefonica.com>
Thu, 7 May 2020 14:34:58 +0000 (14:34 +0000)
Change-Id: Icf8af24c306a5714dbbaa2d71a06cf6f0fd0ca91
Signed-off-by: tierno <alfonso.tiernosepulveda@telefonica.com>
1  2 
RO/osm_ro/openmanod.py
RO/osm_ro/wim/persistence.py

diff --combined RO/osm_ro/openmanod.py
@@@ -53,8 -53,8 +53,8 @@@ import osm_r
  
  __author__ = "Alfonso Tierno, Gerardo Garcia, Pablo Montes"
  __date__ = "$26-aug-2014 11:09:29$"
- __version__ = "7.0.0.post13"
- version_date = "Jan 2019"
 -__version__ = "7.0.1.post26"
++__version__ = "7.1.0.post1"
+ version_date = "May 2020"
  database_version = 40      # expected database schema version
  
  global global_config
@@@ -292,7 -292,7 +292,7 @@@ if __name__ == "__main__"
          logger.critical("Starting openmano server version: '%s %s' command: '%s'",
                          ro_version, version_date, " ".join(sys.argv))
  
-         for log_module in ("nfvo", "http", "vim", "wim", "db", "console", "ovim","sdn"):
+         for log_module in ("nfvo", "http", "vim", "wim", "db", "console", "ovim", "sdn", "sdnconn"):
              log_level_module = "log_level_" + log_module
              log_file_module = "log_file_" + log_module
              logger_module = logging.getLogger('openmano.' + log_module)
@@@ -40,6 -40,7 +40,7 @@@ No domain logic/architectural concern s
  """
  import json
  import logging
+ import string
  from contextlib import contextmanager
  from hashlib import sha1
  from itertools import groupby
@@@ -307,6 -308,11 +308,11 @@@ class WimPersistence(object)
          if "config" in wim_descriptor:
              wim_descriptor["config"] = _serialize(wim_descriptor["config"])
  
+         url = wim_descriptor["wim_url"]
+         wim_descriptor["wim_url"] = url.strip(string.whitespace + "/")
+         # ^  This avoid the common problem caused by trailing spaces/slashes in
+         #    the URL (due to CTRL+C/CTRL+V)
          return self.db.new_row(
              "wims", wim_descriptor, add_uuid=True, confidential_data=True)
  
@@@ -864,7 -870,9 +870,7 @@@ def _preprocess_wim_account(wim_account
      """
      wim_account = preprocess_record(wim_account)
  
 -    created = wim_account.get('created')
 -    wim_account['created'] = (
 -        'true' if created is True or created == 'true' else 'false')
 +    wim_account['sdn'] = False
  
      return wim_account