wip: working out a logger implementation

Author: Prasanna Santhanam

kubeshell/__init__.py

@@ -1 +1,5 @@
 __version__ = '0.0.20'
+from . import logger
+import logging
+import logging.config
+logging.config.dictConfig(logger.loggingConf)

kubeshell/client.py

@@ -1,15 +1,17 @@
 from kubernetes import client, config
 from kubernetes.client.api_client import ApiException
 
+import logging
+
 
 class KubernetesClient(object):
 
     def __init__(self):
+        self.logger = logging.getLogger(__name__)
         try:
             config.load_kube_config()
-        except Exception as e:
-            # TODO: log errors to log file
-            raise(e)
+        except:
+            self.logger.warning("unable to load kube-config")
 
         self.v1 = client.CoreV1Api()
         self.v1Beta1 = client.AppsV1beta1Api()
@@ -25,8 +27,7 @@ def get_resource(self, resource, namespace="all"):
         try:
             ret, namespaced_resource = self._call_api_client(resource)
         except ApiException as ae:
-            # TODO: log warning
-            pass
+            self.logger.warning("resource autocomplete disabled, encountered ApiException %d: %s" % (ae.status, ae.reason))
         if ret:
             for i in ret.items:
                 if namespace == "all" or not namespaced_resource:

kubeshell/logger.py

@@ -0,0 +1,37 @@
+import os
+
+if not os.path.exists(os.path.expanduser("~/.kube-shell")):
+    try:
+        os.makedirs(os.path.expanduser("~/.kube-shell"))
+    except OSError:
+        pass
+
+logfile = os.path.expanduser("~/.kube-shell/debug.log")
+loggingConf = {
+    "version": 1,
+    "disable_existing_loggers": False,
+    "formatters": {
+        "default": {
+            "format": "%(asctime)s %[levelname]-8s %(funcName)s:%(lineno)s - %(message)s",
+        }
+    },
+    "handlers": {
+        "file": {
+            "class": "logging.FileHandler",
+            "level": "ERROR",
+            "formatter": "default",
+            "filename": logfile,
+        }
+    },
+    "loggers": {
+        "kubeshell": {
+            "level": "ERROR",
+            "handlers": ["file"],
+            "propagate": "no",
+        },
+        "": {
+            "level": "ERROR",
+            "handlers": ["file"]
+        }
+    },
+}

kubeshell/main.py

@@ -5,8 +5,13 @@
 from __future__ import print_function, absolute_import, unicode_literals
 from kubeshell.kubeshell import Kubeshell
 
+import logging
+logger = logging.getLogger(__name__)
+
+
 def cli():
     kube_shell= Kubeshell()
+    logger.info("session start")
     kube_shell.run_cli()
 
 if __name__ == "__main__":