Yaniv Bronhaim has submitted this change and it was merged. Change subject: fileSD: Do not try to remove non-existent lease file ......................................................................
fileSD: Do not try to remove non-existent lease file The volume.lease file exists only if a domain has volume leases. Avoid removing the non-existent lease file and the bogus exception in the log when the operation "fails". This is the same patch from master, applied to FileStorageDomain instead of FileStorageDomainManifest, since deleteImage was not moved to the manifest class in 3.6. Change-Id: I525704678707dfef40e34497bc4917dd19d73034 Bug-Url: https://bugzilla.redhat.com/1291207 Signed-off-by: Nir Soffer <nsof...@redhat.com> Reviewed-on: https://gerrit.ovirt.org/50589 Continuous-Integration: Jenkins CI Reviewed-by: Allon Mureinik <amure...@redhat.com> Reviewed-by: Adam Litke <ali...@redhat.com> Reviewed-by: Liron Aravot <lara...@redhat.com> Reviewed-on: https://gerrit.ovirt.org/51132 Reviewed-by: Francesco Romani <from...@redhat.com> Reviewed-by: Freddy Rolland <froll...@redhat.com> Reviewed-by: Idan Shaby <ish...@redhat.com> Reviewed-by: Yaniv Bronhaim <ybron...@redhat.com> --- M vdsm/storage/fileSD.py 1 file changed, 4 insertions(+), 3 deletions(-) Approvals: Nir Soffer: Verified Yaniv Bronhaim: Looks good to me, approved Jenkins CI: Passed CI tests Freddy Rolland: Looks good to me, but someone else must approve Francesco Romani: Looks good to me, but someone else must approve Idan Shaby: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/51132 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I525704678707dfef40e34497bc4917dd19d73034 Gerrit-PatchSet: 3 Gerrit-Project: vdsm Gerrit-Branch: ovirt-3.6 Gerrit-Owner: Nir Soffer <nsof...@redhat.com> Gerrit-Reviewer: Adam Litke <ali...@redhat.com> Gerrit-Reviewer: Francesco Romani <from...@redhat.com> Gerrit-Reviewer: Freddy Rolland <froll...@redhat.com> Gerrit-Reviewer: Idan Shaby <ish...@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Liron Aravot <lara...@redhat.com> Gerrit-Reviewer: Nir Soffer <nsof...@redhat.com> Gerrit-Reviewer: Tal Nisan <tni...@redhat.com> Gerrit-Reviewer: Yaniv Bronhaim <ybron...@redhat.com> Gerrit-Reviewer: gerrit-hooks <automat...@ovirt.org> _______________________________________________ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches