Change in vdsm[master]: supervdsm: Enable multiprocessing module logging.

lyarwood at redhat.com lyarwood at redhat.com
Tue Oct 1 13:29:35 UTC 2013


Lee Yarwood has uploaded a new change for review.

Change subject: supervdsm: Enable multiprocessing module logging.
......................................................................

supervdsm: Enable multiprocessing module logging.

The multiprocessing module includes error logging using Python's logging
module, but it is not enabled by default.  This patch enables the
logging and sets it to propagate to the supervdsmServer's root logger,
so it now defaults to producing logging and can be configured easily
from /etc/vdsm/svdsm.logger.conf.

Change-Id: Idbbe0f1eb294e99de997f78b964a062fdddbb6ea
Signed-off-by: Lee Yarwood <lyarwood at redhat.com>
Signed-off-by: David Gibson <dgibson at redhat.com>
---
M vdsm/supervdsmServer
1 file changed, 2 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/41/19741/1

diff --git a/vdsm/supervdsmServer b/vdsm/supervdsmServer
index 40ec9df..006c35c 100755
--- a/vdsm/supervdsmServer
+++ b/vdsm/supervdsmServer
@@ -30,6 +30,7 @@
 import signal
 import logging
 import logging.config
+import multiprocessing
 
 LOG_CONF_PATH = "/etc/vdsm/svdsm.logger.conf"
 
@@ -356,6 +357,7 @@
 
 def main(sockfile, pidfile=None):
     log = logging.getLogger("SuperVdsm.Server")
+    multiprocessing.get_logger().propagate = 1
 
     def bind(func):
         def wrapper(_SuperVdsm, *args, **kwargs):


-- 
To view, visit http://gerrit.ovirt.org/19741
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Idbbe0f1eb294e99de997f78b964a062fdddbb6ea
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Lee Yarwood <lyarwood at redhat.com>


More information about the vdsm-patches mailing list