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@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/61/35761/1
diff --git a/lib/vdsm/tool/configurators/libvirt.py b/lib/vdsm/tool/configurators/libvirt.py index 1b7556b..d492eab 100644 --- a/lib/vdsm/tool/configurators/libvirt.py +++ b/lib/vdsm/tool/configurators/libvirt.py @@ -481,7 +481,7 @@ 'configure': _prefixAndPrepend, 'prependFile': 'LRCONF_EXAMPLE', 'removeConf': _unprefixAndRemoveSection, - 'persisted': True, + 'persisted': False, },
'LRCONF_EXAMPLE': {
Yaniv Bronhaim has posted comments on this change.
Change subject: Changing persisted value to libvirt logrotate configure ......................................................................
Patch Set 1: Verified+1
Dan Kenigsberg has posted comments on this change.
Change subject: Changing persisted value to libvirt logrotate configure ......................................................................
Patch Set 1: Code-Review+1
Yaniv Bronhaim has posted comments on this change.
Change subject: Changing persisted value to libvirt logrotate configure ......................................................................
Patch Set 1: Code-Review+2
Yaniv Bronhaim has submitted this change and it was merged.
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@redhat.com Reviewed-on: http://gerrit.ovirt.org/35761 Reviewed-by: Dan Kenigsberg danken@redhat.com --- M lib/vdsm/tool/configurators/libvirt.py 1 file changed, 1 insertion(+), 1 deletion(-)
Approvals: Yaniv Bronhaim: Verified; Looks good to me, approved Dan Kenigsberg: Looks good to me, but someone else must approve
oVirt Jenkins CI Server has posted comments on this change.
Change subject: Changing persisted value to libvirt logrotate configure ......................................................................
Patch Set 2:
Build Successful
http://jenkins.ovirt.org/job/vdsm_3.5_create-rpms-fc19-x86_64_merged/128/ : SUCCESS
http://jenkins.ovirt.org/job/vdsm_3.5_create-rpms-fc20-x86_64_merged/128/ : SUCCESS
http://jenkins.ovirt.org/job/vdsm_3.5_create-rpms-el7-x86_64_merged/128/ : SUCCESS
http://jenkins.ovirt.org/job/vdsm_3.5_create-rpms-el6-x86_64_merged/131/ : SUCCESS
vdsm-patches@lists.fedorahosted.org