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 <[email protected]> Gerrit-Reviewer: Francesco Romani <[email protected]> Gerrit-Reviewer: Michal Skrivanek <[email protected]> Gerrit-Reviewer: Yaniv Bronhaim <[email protected]> Gerrit-Reviewer: [email protected] Gerrit-Reviewer: mooli tayer <[email protected]> Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: Yes _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
