[libvirt] [PATCH v2 01/10] virlog: Introduce virLog{Get, Set}DefaultOutput
John Ferlan
jferlan at redhat.com
Thu Dec 8 21:41:23 UTC 2016
On 11/25/2016 08:11 AM, Erik Skultety wrote:
> These helpers will manage the log destination defaults (fetch/set). The reason
> for this is to stay consistent with the current daemon's behaviour with respect
> to /etc/libvirt/<daemon>.conf file, since both assignment of an empty string
> or not setting the log output variable at all trigger the daemon's decision on
> the default log destination which depends on whether the daemon runs daemonized
> or not.
> This patch also changes the logic of the selection of the default
> logging output compared to how it is done now. The main difference though is
> that we should only really care if we're running daemonized or not, disregarding
> the fact of (not) having a TTY completely (introduced by commit eba36a3878) as
> that should be of the libvirtd's parent concern (what FD it will pass to it).
>
> Before:
> if (godaemon || !hasTTY):
> if (journald):
> use journald
>
> if (godaemon):
> if (privileged):
> use SYSCONFIG/libvirtd.log
> else:
> use XDG_CONFIG_HOME/libvirtd.log
> else:
> use stderr
>
> After:
> if (godaemon):
> if (journald):
> use journald
>
> else:
> if (privileged):
> use SYSCONFIG/libvirtd.log
> else:
> use XDG_CONFIG_HOME/libvirtd.log
> else:
> use stderr
>
> Signed-off-by: Erik Skultety <eskultet at redhat.com>
> ---
> src/libvirt_private.syms | 2 ++
> src/util/virlog.c | 92 ++++++++++++++++++++++++++++++++++++++++++++++++
> src/util/virlog.h | 2 ++
> 3 files changed, 96 insertions(+)
>
ACK
John
More information about the libvir-list
mailing list