Piotr Kliczewski has uploaded a new change for review.

Change subject: logging: remove log messages which give little or no value
......................................................................

logging: remove log messages which give little or no value

Some of the messages logged during jsonrpc request processing give no or
little value when diagnosing the issues but are logged quite often.

This patch removes those messages.


Change-Id: If5c6648f39b24adb1a5950c80a3883de0182b3b4
Signed-off-by: pkliczewski <[email protected]>
Bug-Url: https://bugzilla.redhat.com/1239062
---
M lib/yajsonrpc/__init__.py
M lib/yajsonrpc/stompreactor.py
2 files changed, 0 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/20/43720/1

diff --git a/lib/yajsonrpc/__init__.py b/lib/yajsonrpc/__init__.py
index 68af242..778b9c0 100644
--- a/lib/yajsonrpc/__init__.py
+++ b/lib/yajsonrpc/__init__.py
@@ -530,7 +530,6 @@
     @traceback(on=log.name)
     def serve_requests(self):
         while True:
-            self.log.debug("Waiting for request")
             obj = self._workQueue.get()
             if obj is None:
                 break
diff --git a/lib/yajsonrpc/stompreactor.py b/lib/yajsonrpc/stompreactor.py
index e3b71bf..3ed451d 100644
--- a/lib/yajsonrpc/stompreactor.py
+++ b/lib/yajsonrpc/stompreactor.py
@@ -232,7 +232,6 @@
         self._req_dest[request.get("id")] = req_dest
 
     def handle_frame(self, dispatcher, frame):
-        self.log.debug("Handling message %s", frame)
         try:
             self._commands[frame.command](dispatcher, frame)
         except KeyError:


-- 
To view, visit https://gerrit.ovirt.org/43720
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: If5c6648f39b24adb1a5950c80a3883de0182b3b4
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Piotr Kliczewski <[email protected]>
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to