Avoid huge libvirtd.log files on Ubuntu hosts
The libvirt log filter settings match against the filename of the libvirt source emitting the log message. Normally these file names are relative to the source tree root, but in the Ubuntu binary packages these have somehow ended up as absolute filenames from the OS root. This means that a log filter of '1:libvirt' which is only intended to match src/libvirt.c will in fact match every single file. This caused enourmous log files on Ubuntu hosts running the gate. The fix is to use '1:libvirt.c' as a more specific filename match, but we can't do this unconditionally because libvirt >= 1.2.3 does not use filenames for log filter matching anymore. So only change the match on Ubuntu hosts for now, since that's where the original problem lies. While doing this, also turn off the logging of object ref and unref operations, since those pollute the logs with lots of noise. Change-Id: I71b67507a4e68a7bff0c358857aaaac08ef0c420
This commit is contained in:
parent
c06c9e1f90
commit
a12f996778
@ -112,7 +112,15 @@ EOF
|
||||
|
||||
# Enable server side traces for libvirtd
|
||||
if [[ "$DEBUG_LIBVIRT" = "True" ]] ; then
|
||||
local log_filters="1:libvirt 1:qemu 1:conf 1:security 3:event 3:json 3:file 1:util"
|
||||
if is_ubuntu; then
|
||||
# Unexpectedly binary package builds in ubuntu get fully qualified
|
||||
# source file paths, not relative paths. This screws with the matching
|
||||
# of '1:libvirt' making everything turn on. So use libvirt.c for now.
|
||||
# This will have to be re-visited when Ubuntu ships libvirt >= 1.2.3
|
||||
local log_filters="1:libvirt.c 1:qemu 1:conf 1:security 3:object 3:event 3:json 3:file 1:util"
|
||||
else
|
||||
local log_filters="1:libvirt 1:qemu 1:conf 1:security 3:object 3:event 3:json 3:file 1:util"
|
||||
fi
|
||||
local log_outputs="1:file:/var/log/libvirt/libvirtd.log"
|
||||
if ! grep -q "log_filters=\"$log_filters\"" /etc/libvirt/libvirtd.conf; then
|
||||
echo "log_filters=\"$log_filters\"" | sudo tee -a /etc/libvirt/libvirtd.conf
|
||||
|
Loading…
Reference in New Issue
Block a user