Dan Kenigsberg 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 <ybron...@redhat.com> Reviewed-on: http://gerrit.ovirt.org/35758 Reviewed-by: Dan Kenigsberg <dan...@redhat.com> --- M lib/vdsm/tool/configurators/libvirt.py 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Yaniv Bronhaim: Verified Dan Kenigsberg: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/35758 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3f1c4d5d513e8699543727fff9f30a8213c66bfd Gerrit-PatchSet: 2 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Yaniv Bronhaim <ybron...@redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alo...@redhat.com> Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com> Gerrit-Reviewer: Saggi Mizrahi <smizr...@redhat.com> Gerrit-Reviewer: Yaniv Bronhaim <ybron...@redhat.com> Gerrit-Reviewer: automat...@ovirt.org Gerrit-Reviewer: mooli tayer <mta...@redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches