Change in vdsm[ovirt-3.6]: storage: Clean up image links during teardownImage

nsoffer at redhat.com nsoffer at redhat.com
Wed Mar 23 22:52:04 UTC 2016


Nir Soffer has posted comments on this change.

Change subject: storage: Clean up image links during teardownImage
......................................................................


Patch Set 2:

(1 comment)

https://gerrit.ovirt.org/#/c/55164/2//COMMIT_MSG
Commit Message:

Line 10: /var/run/vdsm/images/... Unfortunately on block domains we are leaving
Line 11: around a broken symlink in /rhev/data-center/... in some cases and this
Line 12: will cause storage operations related to a disk of a VM that was
Line 13: previously running to fail on this host.  The solution is to also remove
Line 14: this broken symlink during teardownImage.
> This fix is not enough, we can still have broken links if vdsm is killed in
See this draft https://gerrit.ovirt.org/55175 for the additional fix.
Line 15: 
Line 16: Change-Id: I5fb871f3ccd785c555c72db34ab818c04d000534
Line 17: Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1319987


-- 
To view, visit https://gerrit.ovirt.org/55164
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I5fb871f3ccd785c555c72db34ab818c04d000534
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-3.6
Gerrit-Owner: Adam Litke <alitke at redhat.com>
Gerrit-Reviewer: Adam Litke <alitke at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Idan Shaby <ishaby at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Liron Aravot <laravot at redhat.com>
Gerrit-Reviewer: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>
Gerrit-HasComments: Yes


More information about the vdsm-patches mailing list