Change in vdsm[master]: logging: Enable all storage loggers

ybronhei at redhat.com ybronhei at redhat.com
Wed Mar 5 15:21:52 UTC 2014


Yaniv Bronhaim has posted comments on this change.

Change subject: logging: Enable all storage loggers
......................................................................


Patch Set 1:

it is required and important! using a logger that doesn't exist is a bug, even if now unknown logger will be caught by root handler. if one day we will want to split all logs from storage logs or change only storage related logs handling, we'll have the option. utils logger is redundant, use vds handler if its not related to storage.

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I1dd8cd29377b12dc290f90b7c6bf314d5624a830
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: Tomáš Došek <tdosek at redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No


More information about the vdsm-patches mailing list