Nir Soffer has posted comments on this change. Change subject: fileSD: fix deleteImage flow ......................................................................
Patch Set 1: (1 comment) https://gerrit.ovirt.org/#/c/50804/1//COMMIT_MSG Commit Message: Line 14: Line 15: Also, if one of the files couldn't be removed, we didn't try to remove Line 16: the others. Line 17: Now we handle each removal separately, so that a failure in the removal Line 18: of one file doesn't affect the removal of the others. Please describe also fixing handling of missing files, logging a warning instead of useless a traceback. Line 19: Line 20: Bug-Url: https://bugzilla.redhat.com/1292092 Line 21: Change-Id: I9eeb28a70f708a4f9a5effe4ff294da63b757369 -- To view, visit https://gerrit.ovirt.org/50804 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I9eeb28a70f708a4f9a5effe4ff294da63b757369 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Idan Shaby <[email protected]> Gerrit-Reviewer: Adam Litke <[email protected]> Gerrit-Reviewer: Allon Mureinik <[email protected]> Gerrit-Reviewer: Freddy Rolland <[email protected]> Gerrit-Reviewer: Idan Shaby <[email protected]> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Nir Soffer <[email protected]> Gerrit-Reviewer: Tal Nisan <[email protected]> Gerrit-Reviewer: gerrit-hooks <[email protected]> Gerrit-HasComments: Yes _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
