Change in vdsm[master]: vdsm-tool: always define log_outputs in libvirt's conf.

mtayer at redhat.com mtayer at redhat.com
Tue Sep 16 15:08:00 UTC 2014


mooli tayer has uploaded a new change for review.

Change subject: vdsm-tool: always define log_outputs in libvirt's conf.
......................................................................

vdsm-tool: always define log_outputs in libvirt's conf.

According to [1] libvirt will use for logging:
systemd journal, if /run/systemd/journal/socket exists.
file /var/log/libvirt/libvirtd.log if running as a daemon.

Turns out if both are true, the first will apply.

This patch reverts part of previews [2]

[1] http://libvirt.org/logging.html
[2] http://gerrit.ovirt.org/#/q/I251c2534d3284aa95bffa56143c449d5de887008,n,z

Change-Id: Ic839244d7d4c85d56275f07b65feeef4ec6176ec
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=114176
Signed-off-by: Mooli Tayer <mtayer at redhat.com>
---
M lib/vdsm/tool/configurators/libvirt.py
1 file changed, 2 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/91/32991/1

diff --git a/lib/vdsm/tool/configurators/libvirt.py b/lib/vdsm/tool/configurators/libvirt.py
index 5d15de2..1cab605 100644
--- a/lib/vdsm/tool/configurators/libvirt.py
+++ b/lib/vdsm/tool/configurators/libvirt.py
@@ -339,6 +339,8 @@
                         'auth_unix_rw': '"sasl"',
                         'host_uuid': '"' + str(uuid.uuid4()) + '"',
                         'keepalive_interval': -1,
+                        'log_outputs': (
+                            '"1:file:/var/log/libvirt/libvirtd.log"'),
                     },
                 },
                 {


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ic839244d7d4c85d56275f07b65feeef4ec6176ec
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: mooli tayer <mtayer at redhat.com>


More information about the vdsm-patches mailing list