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(a)redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce(a)redhat.com>
Gerrit-Reviewer: Nir Soffer <nsoffer(a)redhat.com>
Gerrit-Reviewer: Tomáš Došek <tdosek(a)redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei(a)redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No