Change in vdsm[master]: vdsmd.init: ajust libvirt log file

danken at redhat.com danken at redhat.com
Sat May 18 14:14:11 UTC 2013


Dan Kenigsberg has posted comments on this change.

Change subject: vdsmd.init: ajust libvirt log file
......................................................................


Patch Set 1: (1 inline comment)

....................................................
File vdsm/vdsmd.init.in
Line 391:     local llogr=/etc/logrotate.d/libvirtd
Line 392:     local stanza=`mktemp`
Line 393:     /bin/cat > "$stanza" <<EOF
Line 394: # vdsm
Line 395: /var/log/libvirt/libvirtd.log {
I have a vague memory that having two /var/log/libvirt/libvirtd.log keys in this file, is making logrotate cry or otherwise confused. Please double check this issue. We may need to have this file more painfully edited. Maybe, just maybe, we should drop the default stanza.
Line 396:     rotate 100
Line 397:     missingok
Line 398:     copytruncate
Line 399:     size 15M


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Iee325d456cc3d184c0084c4bf547bc126a00938a
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Douglas Schilling Landgraf <dougsland at redhat.com>
Gerrit-Reviewer: Barak Azulay <bazulay at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server


More information about the vdsm-patches mailing list