Nir Soffer has submitted this change and it was merged. Change subject: blockVolume: Remove stale links to image directory ......................................................................
blockVolume: Remove stale links to image directory Since we started to create the image directory under "/run/vdsm", the image directory under "/rhev/data-center/pool/sd/images/" may be a link to "/run/vdsm/storage/sd/image". The old code (from rhev-3.0) creating the image directory lazily was not designed to handle links and was happily using a stale link instead of creating the image directory. Now we detect and remove stale links, so proper image directory is created. Since stale links are not expected in normal usage, we log a warning in this case. Change-Id: Ic694caab0e030c675e4a3a0c0638298abfea1a8c Bug-Url: https://bugzilla.redhat.com/1319987 Signed-off-by: Nir Soffer <nsof...@redhat.com> Signed-off-by: Adam Litke <ali...@redhat.com> Reviewed-on: https://gerrit.ovirt.org/55175 Continuous-Integration: Jenkins CI --- M vdsm/storage/blockVolume.py 1 file changed, 8 insertions(+), 0 deletions(-) Approvals: Nir Soffer: Looks good to me, approved Adam Litke: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/55175 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic694caab0e030c675e4a3a0c0638298abfea1a8c Gerrit-PatchSet: 3 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Nir Soffer <nsof...@redhat.com> Gerrit-Reviewer: Adam Litke <ali...@redhat.com> Gerrit-Reviewer: Allon Mureinik <amure...@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Nir Soffer <nsof...@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