Change in vdsm[master]: vdsmd.init: replace libvirt logs filter

danken at redhat.com danken at redhat.com
Wed Apr 24 07:02:35 UTC 2013


Dan Kenigsberg has posted comments on this change.

Change subject: vdsmd.init: replace libvirt logs filter
......................................................................


Patch Set 3: (1 inline comment)

....................................................
File vdsm/vdsmd.init.in
Line 309:     set_if_default $qconf save_image_format \"lzop\"
Line 310:     # FIXME until we are confident with libvirt integration, let us have a verbose log
Line 311:     set_if_default $lconf log_outputs \"1:file:/var/log/libvirtd.log\"
Line 312:     set_if_default $lconf log_filters "\"3:virobject 3:virfile 2:virnetlink \
Line 313: 3:cgroup 3:event 3:json 1:libvirt 1:util 1:qemu\""
Pardon my ignorance, but what are all these new log sources that you add, and why 
adding them *reduces* the log size?
Line 314: 
Line 315:     # If the ssl flag is set, update the libvirt and qemu configuration files
Line 316:     # with the location for certificates and permissions.
Line 317:     if [ -f $ts/certs/cacert.pem -a \


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I31218b3a6ac2f0866df03c6f84c9303d1a53f745
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Douglas Schilling Landgraf <dougsland at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland at redhat.com>
Gerrit-Reviewer: Pavel Zhukov <pzhukov at redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server


More information about the vdsm-patches mailing list