X-Git-Url: https://gerrit.opnfv.org/gerrit/gitweb?a=blobdiff_plain;f=nfvbench%2Fconfig.py;h=4cc9c8686914c636d0c0b456193cf86eece566a1;hb=95f2491ed89ac99b0d8bd006b4a13cbeb1eb96ce;hp=2ed726dd667b2caacda2ab243db061084af49931;hpb=fd72e102b0d6b6b62f72a06ad950e7959c54d7ef;p=nfvbench.git

diff --git a/nfvbench/config.py b/nfvbench/config.py
index 2ed726d..4cc9c86 100644
--- a/nfvbench/config.py
+++ b/nfvbench/config.py
@@ -14,11 +14,11 @@
 #
 
 from attrdict import AttrDict
-from log import LOG
 import yaml
 
+from .log import LOG
 
-def config_load(file_name, from_cfg=None, whitelist_keys=[]):
+def config_load(file_name, from_cfg=None, whitelist_keys=None):
     """Load a yaml file into a config dict, merge with from_cfg if not None
     The config file content taking precedence in case of duplicate
     """
@@ -31,29 +31,40 @@ def config_load(file_name, from_cfg=None, whitelist_keys=[]):
                         .format(file_name))
 
     if from_cfg:
+        if not whitelist_keys:
+            whitelist_keys = []
         _validate_config(cfg, from_cfg, whitelist_keys)
         cfg = from_cfg + cfg
 
     return cfg
 
 
-def config_loads(cfg_text, from_cfg=None, whitelist_keys=[]):
+def config_loads(cfg_text, from_cfg=None, whitelist_keys=None):
     """Same as config_load but load from a string
     """
     try:
-        cfg = AttrDict(yaml.load(cfg_text))
+        cfg = AttrDict(yaml.safe_load(cfg_text))
     except TypeError:
         # empty string
         cfg = AttrDict()
+    except ValueError as e:
+        # In case of wrong path or file not readable or string not well formatted
+        LOG.error("String %s is not well formatted. Please verify your yaml/json string. "
+                  "If string is a file path, file was not found. Please use correct path and "
+                  "verify it is visible to container if you run nfvbench in container.", cfg_text)
+        raise Exception(e)
     if from_cfg:
+        if not whitelist_keys:
+            whitelist_keys = []
         _validate_config(cfg, from_cfg, whitelist_keys)
         return from_cfg + cfg
     return cfg
 
+
 def _validate_config(subset, superset, whitelist_keys):
     def get_err_config(subset, superset):
         result = {}
-        for k, v in subset.items():
+        for k, v in list(subset.items()):
             if k not in whitelist_keys:
                 if k not in superset:
                     result.update({k: v})