Change in vdsm[master]: logs: improve VDSM logs human-readability
ykleinbe at redhat.com
ykleinbe at redhat.com
Wed Jun 18 21:50:08 UTC 2014
Yoav Kleinberger has posted comments on this change.
Change subject: logs: improve VDSM logs human-readability
......................................................................
Patch Set 1:
(1 comment)
http://gerrit.ovirt.org/#/c/28869/1/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: %(message)s (in function %(funcName)s %(pathname)s:%(lineno)d)
> Can you explain the benefits of the suggested format string, beside persona
* the %(name)s is what made the previous format so bad. It broke the column-like structure that my proposition has, which makes everything look more tidy. The filename at the end essentially provides the same information - even better.
* the :: in my opinion is OK, as long as there's just ONE :: and not many of them. I am not a C++ compiler.
* the filename:line at the end helps keep a more table-like structure, which I think is the main advantage of the new format.
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: 1
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: Liron Ar <laravot at redhat.com>
Gerrit-Reviewer: Maor Lipchuk <mlipchuk at redhat.com>
Gerrit-Reviewer: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: Vered Volansky <vvolansk 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