Changed logging messages to debug (#60)

* Changed logging messages to debug

* Added pylint validation to travis

Author: avara1986

pylintrc

@@ -54,7 +54,7 @@ confidence=
 # --enable=similarities". If you want to run only the classes checker, but have
 # no Warning level messages displayed, use"--disable=all --enable=classes
 # --disable=W"
-disable=C0301,C0111,C0103,logging-format-interpolation
+disable=C0301,C0111,C0103,logging-format-interpolation,broad-except,unnecessary-pass
 
 # Enable the message, report, category or checker with the given id(s). You can
 # either give multiple identifier separated by comma (,) or put this option
@@ -362,7 +362,7 @@ dummy-variables-rgx=_+$|(_[a-zA-Z0-9_]*[a-zA-Z0-9]+?$)|dummy|^ignored_|^unused_
 
 # Argument names that match this expression will be ignored. Default to name
 # with leading underscore
-ignored-argument-names=_.*|^ignored_|^unused_
+ignored-argument-names=arg|args|kwargs|_.*|^ignored_|^unused_
 
 # Tells whether we should check for unused import in __init__ files.
 init-import=no

pyms/config/confile.py

@@ -45,7 +45,8 @@ def normalize_config(self, config):
                 item = ConfFile(config=item, empty_init=self.empty_init)
             yield self.normalize_keys(key), item
 
-    def normalize_keys(self, key):
+    @staticmethod
+    def normalize_keys(key):
         """The keys will be transformed to a attribute. We need to replace the charactes not valid"""
         key = key.replace("-", "_")
         return key
@@ -65,18 +66,18 @@ def __getattr__(self, name, *args, **kwargs):
         except KeyError:
             if self.empty_init:
                 return ConfFile(config={}, empty_init=self.empty_init)
-            else:
-                raise AttrDoesNotExistException("Variable {} not exist in the config file".format(name))
+            raise AttrDoesNotExistException("Variable {} not exist in the config file".format(name))
 
     def _get_conf_from_env(self):
         config_file = os.environ.get(CONFIGMAP_FILE_ENVIRONMENT, self.default_file)
-        logger.info("[CONF] Searching file in ENV[{}]: {}...".format(CONFIGMAP_FILE_ENVIRONMENT, config_file))
+        logger.debug("[CONF] Searching file in ENV[{}]: {}...".format(CONFIGMAP_FILE_ENVIRONMENT, config_file))
         return self._get_conf_from_file(config_file)
 
-    def _get_conf_from_file(self, path: Text) -> dict:
+    @staticmethod
+    def _get_conf_from_file(path: Text) -> dict:
         if not path or not os.path.isfile(path):
             return {}
-        logger.info("[CONF] Configmap {} found".format(path))
+        logger.debug("[CONF] Configmap {} found".format(path))
         conf = anyconfig.load(path)
         return conf
 

pyms/flask/services/driver.py

@@ -32,5 +32,5 @@ def get_services(self):
 
     def get_service(self, service, *args, **kwargs):
         service_object = import_from("pyms.flask.services.{}".format(service), "Service")
-        logger.info("Init service {}".format(service))
+        logger.debug("Init service {}".format(service))
         return service_object(service=self.service, *args, **kwargs)

pyms/flask/services/requests.py

@@ -156,7 +156,7 @@ def get(self, url, path_params=None, params=None, headers=None, propagate_header
 
         full_url = self._build_url(url, path_params)
         headers = self._get_headers(headers=headers, propagate_headers=propagate_headers)
-        logger.info("Get with url {}, params {}, headers {}, kwargs {}".
+        logger.debug("Get with url {}, params {}, headers {}, kwargs {}".
                     format(full_url, params, headers, kwargs))
 
         session = requests.Session()
@@ -197,12 +197,12 @@ def post(self, url, path_params=None, data=None, json=None, headers=None, **kwar
 
         full_url = self._build_url(url, path_params)
         headers = self._get_headers(headers)
-        logger.info("Post with url {}, data {}, json {}, headers {}, kwargs {}".format(full_url, data, json,
+        logger.debug("Post with url {}, data {}, json {}, headers {}, kwargs {}".format(full_url, data, json,
                                                                                        headers, kwargs))
 
         session = requests.Session()
         response = self.requests(session=session).post(full_url, data=data, json=json, headers=headers, **kwargs)
-        logger.info("Response {}".format(response))
+        logger.debug("Response {}".format(response))
 
         return response
 
@@ -240,12 +240,12 @@ def put(self, url, path_params=None, data=None, headers=None, **kwargs):
 
         full_url = self._build_url(url, path_params)
         headers = self._get_headers(headers)
-        logger.info("Put with url {}, data {}, headers {}, kwargs {}".format(full_url, data, headers,
+        logger.debug("Put with url {}, data {}, headers {}, kwargs {}".format(full_url, data, headers,
                                                                              kwargs))
 
         session = requests.Session()
         response = self.requests(session=session).put(full_url, data, headers=headers, **kwargs)
-        logger.info("Response {}".format(response))
+        logger.debug("Response {}".format(response))
 
         return response
 
@@ -280,10 +280,10 @@ def delete(self, url, path_params=None, headers=None, **kwargs):
 
         full_url = self._build_url(url, path_params)
         headers = self._get_headers(headers)
-        logger.info("Delete with url {}, headers {}, kwargs {}".format(full_url, headers, kwargs))
+        logger.debug("Delete with url {}, headers {}, kwargs {}".format(full_url, headers, kwargs))
 
         session = requests.Session()
         response = self.requests(session=session).delete(full_url, headers=headers, **kwargs)
-        logger.info("Response {}".format(response))
+        logger.debug("Response {}".format(response))
 
         return response

tox.ini

@@ -22,7 +22,7 @@ basepython = python3.6
 commands = safety check -r requirements-tests.txt
 [testenv:pylint]
 basepython = python3.6
-commands = pylint --rcfile={toxinidir}/pylintrc {toxinidir}/pyms --exit-zero
+commands = pylint --rcfile={toxinidir}/pylintrc {toxinidir}/pyms
 [testenv:docs]
 basepython = python3.6
 skipsdist = True