Change in vdsm[master]: libvirtd.log: time to be less verbose

fromani at redhat.com fromani at redhat.com
Fri Aug 22 11:45:53 UTC 2014


Francesco Romani has posted comments on this change.

Change subject: libvirtd.log: time to be less verbose
......................................................................


Patch Set 2: Code-Review-1

(2 comments)

Still like the idea, but I believe we can discuss some tuning.

-1 only for visibility

http://gerrit.ovirt.org/#/c/31135/2/lib/vdsm/tool/configurator.py
File lib/vdsm/tool/configurator.py:

Line 390
Line 391
Line 392
Line 393
Line 394
what about just changing to

'log_outputs': (
   '"3:file:/var/log/libvirt/libvirtd.log"'),

this will severly reduce the log pressure but will preserve the log file, which I like to have.


Line 394
Line 395
Line 396
Line 397
Line 398
OK with deleting this to reduce the noise


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I251c2534d3284aa95bffa56143c449d5de887008
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek at redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: mooli tayer <mtayer at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes


More information about the vdsm-patches mailing list