Logging enhancement for openo vmlist API

Change-Id: I27bc6aaa7d6f525e2716df35301bdf2d9b63b307
Issue-ID: MULTICLOUD-178
Signed-off-by: Yun Huang <yun.huang@windriver.com>

diff --git a/share/newton_base/openoapi/vmlist.py b/share/newton_base/openoapi/vmlist.py
index 8d2e0c6..6150b86 100755 (executable)
--- a/share/newton_base/openoapi/vmlist.py
+++ b/share/newton_base/openoapi/vmlist.py
@@ -38,12 +38,17 @@ class VMlist(APIView):
     ]\r
 \r
     def get(self, request, vimid="", tenantid="", serverid=""):\r
-        logger.debug("servers--get::> %s" % request.data)\r
+        logger.info("vimid, tenantid, flavorid = %s,%s,%s" % (vimid, tenantid, flavorid))\r
+        if request.data:\r
+            logger.debug("With data = %s" % request.data)\r
+            pass\r
         try:\r
             query = VimDriverUtils.get_query_part(request)\r
             content, status_code = self.get_servers(query, vimid, tenantid, serverid)\r
+            logger.info("response with status = %s" % status_code)\r
             return Response(data=content, status=status_code)\r
         except VimDriverNewtonException as e:\r
+            logger.error("response with status = %s" % e.status_code)\r
             return Response(data={'error': e.content}, status=e.status_code)\r
         except HttpError as e:\r
             logger.error("HttpError: status:%s, response:%s" % (e.http_status, e.response.json()))\r
@@ -53,15 +58,20 @@ class VMlist(APIView):
             return Response(data={'error': str(e)},\r
                             status=status.HTTP_500_INTERNAL_SERVER_ERROR)\r
 \r
-                                                       \r
     def get_servers(self, query="", vimid="", tenantid="", serverid=""):\r
-        logger.debug("servers--get_servers::> %s" % serverid)\r
 \r
         req_resouce = "/servers"\r
         vim = VimDriverUtils.get_vim_info(vimid)\r
         vim["domain"] = "Default"\r
         sess = VimDriverUtils.get_session(vim, tenantid)\r
+\r
+        logger.info("making request with URI:%s" % req_resouce)\r
         resp = sess.get(req_resouce, endpoint_filter=self.service)\r
+        logger.info("request returns with status %s" % resp.status_code)\r
+        if resp.status_code == status.HTTP_200_OK:\r
+            logger.debug("with content:%s" % resp.json())\r
+            pass\r
+\r
         content = resp.json()\r
         vim_dict = {\r
             "vimName": vim["name"],\r
@@ -71,4 +81,4 @@ class VMlist(APIView):
 \r
         return content, resp.status_code\r
 \r
-    \r
+\r