Change in vdsm[ovirt-3.5]: oop: Use Storage.oop logger

ykaplan at redhat.com ykaplan at redhat.com
Mon Sep 22 09:46:00 UTC 2014


Hello Nir Soffer, Saggi Mizrahi, mooli tayer, Dima Kuznetsov, Dan Kenigsberg,

I'd like you to do a code review.  Please visit

    http://gerrit.ovirt.org/33163

to review the following change.

Change subject: oop: Use Storage.oop logger
......................................................................

oop: Use Storage.oop logger

Use storage logger to avoid oop logs in /var/log/messages.
Storage logger is configured to not send messages to syslog.

Change-Id: Id2d0474d1d69e227aa12b8b70022930f66e7387d
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1131867
Signed-off-by: Yeela Kaplan <ykaplan at redhat.com>
Reviewed-on: http://gerrit.ovirt.org/31908
Reviewed-by: Dima Kuznetsov <dkuznets at redhat.com>
Reviewed-by: mooli tayer <mtayer at redhat.com>
Reviewed-by: Saggi Mizrahi <smizrahi at redhat.com>
Reviewed-by: Nir Soffer <nsoffer at redhat.com>
Reviewed-by: Dan Kenigsberg <danken at redhat.com>
---
M vdsm/storage/outOfProcess.py
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/63/33163/1

diff --git a/vdsm/storage/outOfProcess.py b/vdsm/storage/outOfProcess.py
index 6230fad..17445e6 100644
--- a/vdsm/storage/outOfProcess.py
+++ b/vdsm/storage/outOfProcess.py
@@ -50,7 +50,7 @@
 _procLock = threading.Lock()
 _proc = {}
 
-log = logging.getLogger('oop')
+log = logging.getLogger('Storage.oop')
 
 
 def setDefaultImpl(impl):


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Id2d0474d1d69e227aa12b8b70022930f66e7387d
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-3.5
Gerrit-Owner: Yeela Kaplan <ykaplan at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Dima Kuznetsov <dkuznets at redhat.com>
Gerrit-Reviewer: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: Saggi Mizrahi <smizrahi at redhat.com>
Gerrit-Reviewer: mooli tayer <mtayer at redhat.com>


More information about the vdsm-patches mailing list