Change in vdsm[master]: Changing persisted value to libvirt logrotate configure

ybronhei at redhat.com ybronhei at redhat.com
Mon Dec 1 15:26:30 UTC 2014


Yaniv Bronhaim has uploaded a new change for review.

Change subject: Changing persisted value to libvirt logrotate configure
......................................................................

Changing persisted value to libvirt logrotate configure

Setting persisted to false means that is-configured won't check the
status of this file. This to avoid differences between ovirt-3.4
configuration to ovirt-3.5 in /etc/logrotate.d/libvirtd

In ovirt-3.4 we set the prefix of the configuration without the conf
version, therefore in 3.5 we alerted that libvirt is not configured
and asked to run vdsm-tool configure all-over again without having any
differences between the versions.

Change-Id: I3f1c4d5d513e8699543727fff9f30a8213c66bfd
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1168695
Signed-off-by: Yaniv Bronhaim <ybronhei at redhat.com>
---
M lib/vdsm/tool/configurators/libvirt.py
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/58/35758/1

diff --git a/lib/vdsm/tool/configurators/libvirt.py b/lib/vdsm/tool/configurators/libvirt.py
index 796bc9e..35ae5d4 100644
--- a/lib/vdsm/tool/configurators/libvirt.py
+++ b/lib/vdsm/tool/configurators/libvirt.py
@@ -482,7 +482,7 @@
             'configure': _prefixAndPrepend,
             'prependFile': 'LRCONF_EXAMPLE',
             'removeConf': _unprefixAndRemoveSection,
-            'persisted': True,
+            'persisted': False,
         },
 
         'LRCONF_EXAMPLE': {


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I3f1c4d5d513e8699543727fff9f30a8213c66bfd
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Yaniv Bronhaim <ybronhei at redhat.com>


More information about the vdsm-patches mailing list