Nir Soffer has posted comments on this change. Change subject: logging: Enable all storage loggers ......................................................................
Patch Set 1: This can help to configure the new loggers that were missing before. For example, some logger may log to syslog while storage logger are configured to not log to syslog. Or maybe you want to change the configuration for all storage loggers, now all sub loggers inherit the storage configuration. However I think that some loggers, like ResourceMonitor are not related to storage and should sub loggeres of utils logger (which does not exists yet). -- 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 <nsof...@redhat.com> Gerrit-Reviewer: Allon Mureinik <amure...@redhat.com> Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com> Gerrit-Reviewer: Federico Simoncelli <fsimo...@redhat.com> Gerrit-Reviewer: Nir Soffer <nsof...@redhat.com> Gerrit-Reviewer: Tomáš Došek <tdo...@redhat.com> Gerrit-Reviewer: Yaniv Bronhaim <ybron...@redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: No _______________________________________________ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches