Change in vdsm[master]: libvirt_configure: persist cmd qemu-sanlock.conf
danken at redhat.com
danken at redhat.com
Sat Mar 22 15:38:00 UTC 2014
Dan Kenigsberg has submitted this change and it was merged.
Change subject: libvirt_configure: persist cmd qemu-sanlock.conf
......................................................................
libvirt_configure: persist cmd qemu-sanlock.conf
On a ovirt-node if executing vdsm-tool configure --force
with libvirt sanlock enabled it will set /etc/libvirt/qemu-sanlock.conf
but won't use persist command so next reboot the changes will be lost
and vdsm-tool will complain about libvirt not configured during vdsm start.
Change-Id: I299f5cb7af4d2e09208835aeda28ec1f3c3def7d
Signed-off-by: Douglas Schilling Landgraf <dougsland at redhat.com>
Reviewed-on: http://gerrit.ovirt.org/25968
Reviewed-by: Fabian Deutsch <fabiand at fedoraproject.org>
Reviewed-by: Dan Kenigsberg <danken at redhat.com>
---
M lib/vdsm/tool/libvirt_configure.sh.in
1 file changed, 6 insertions(+), 1 deletion(-)
Approvals:
Fabian Deutsch: Looks good to me, but someone else must approve
Douglas Schilling Landgraf: Verified
Dan Kenigsberg: Looks good to me, approved
--
To view, visit http://gerrit.ovirt.org/25968
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I299f5cb7af4d2e09208835aeda28ec1f3c3def7d
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Douglas Schilling Landgraf <dougsland at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland at redhat.com>
Gerrit-Reviewer: Fabian Deutsch <fabiand at fedoraproject.org>
Gerrit-Reviewer: Kiril Nesenko <knesenko at redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
More information about the vdsm-patches
mailing list