Change in vdsm[ovirt-3.6]: fileSD: improve error handling in deleteImage

fromani at redhat.com fromani at redhat.com
Mon Jan 4 15:03:28 UTC 2016


Francesco Romani has posted comments on this change.

Change subject: fileSD: improve error handling in deleteImage
......................................................................


Patch Set 1:

(1 comment)

codewise seems ok, and has ACKs from storage devs. I have a question about backport process, trying to streamline it as much as we can.

https://gerrit.ovirt.org/#/c/51299/1//COMMIT_MSG
Commit Message:

Line 19: the others.  Now we handle each removal separately, so that a failure in
Line 20: the removal of one file doesn't affect the removal of the others.
Line 21: 
Line 22: This patch is a backport for patch
Line 23: I9eeb28a70f708a4f9a5effe4ff294da63b757369 which was reverted.
what happens if you use this change-id? Does gerrit refuse the patch?
Line 24: 
Line 25: Change-Id: I22ceee059cce1ec65cd455ce567cba0e49c00db2
Line 26: Bug-Url: https://bugzilla.redhat.com/1292092
Line 27: Signed-off-by: Idan Shaby <ishaby at redhat.com>


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I22ceee059cce1ec65cd455ce567cba0e49c00db2
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-3.6
Gerrit-Owner: Idan Shaby <ishaby at redhat.com>
Gerrit-Reviewer: Adam Litke <alitke at redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Idan Shaby <ishaby at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>
Gerrit-HasComments: Yes


More information about the vdsm-patches mailing list