Change in vdsm[master]: logs: improve VDSM logs human-readability

nsoffer at redhat.com nsoffer at redhat.com
Thu Jun 26 06:07:23 UTC 2014


Nir Soffer has posted comments on this change.

Change subject: logs: improve VDSM logs human-readability
......................................................................


Patch Set 4:

(1 comment)

http://gerrit.ovirt.org/#/c/28869/4/vdsm/logger.conf.in
File vdsm/logger.conf.in:

Line 66: [formatter_none]
Line 67: format: %(message)s
Line 68: 
Line 69: [formatter_long]
Line 70: format: %(asctime)s %(levelname)s %(threadName)s [%(name)s ::%(module)s in function %(funcName)s %(pathname)s:%(lineno)d]: %(message)s
> Only redundant if you know the location of each module by heart. Sure, you 
I agree with Francesco - this log is not meant to make life easier for new developers but make it easy for people who work with vdsm log daily.
Line 71: 
Line 72: [formatter_sysform]
Line 73: format= vdsm %(name)s %(levelname)s %(message)s
Line 74: datefmt=


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I65fcc843b4c2c6f1b0f438937a0702b80b714978
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Yoav Kleinberger <ykleinbe 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: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Liron Aravot <laravot at redhat.com>
Gerrit-Reviewer: Maor Lipchuk <mlipchuk at redhat.com>
Gerrit-Reviewer: Marina Kalinin <marinamku at gmail.com>
Gerrit-Reviewer: Martin Sivák <msivak at redhat.com>
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek at redhat.com>
Gerrit-Reviewer: Michal Skrivanek <mskrivan at redhat.com>
Gerrit-Reviewer: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: Saggi Mizrahi <smizrahi at redhat.com>
Gerrit-Reviewer: Vered Volansky <vvolansk at redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: Yoav Kleinberger <ykleinbe at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes


More information about the vdsm-patches mailing list