On 03/07/2019 08.03, Peter Krempa wrote: > I'm not sure that this is the right thing to do. virtlogd has some > internal log rotation mechanisms
logrotate is already in use here and this patch doesn't change what is rotated nor how often it is rotated. > and also the SIGUSR1 action is reserved > for re-exec updates and not for dealing with modified files The mechanism (re-exec) and implementation details for both uses do not conflict. > Additionally the internals are supposed to keep all the FD's open during > re-exec so I'm not even sure that this will help. This test shows that it helps: # ls -la /var/log/libvirt/qemu/foo.log -rw------- 1 root root 0 Jun 30 00:00 /var/log/libvirt/qemu/foo.log # mv /var/log/libvirt/qemu/foo.log /var/log/libvirt/qemu/foo.log.0 # /usr/bin/killall -USR1 virtlogd # ls -la /var/log/libvirt/qemu/foo.log -rw------- 1 root root 0 Jul 3 08:06 /var/log/libvirt/qemu/foo.log -- Best regards, Jan Zerebecki -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list
