Dan Kenigsberg has submitted this change and it was merged. Change subject: hsm: deleteImage - lock image by imgUUID and sdUUID ......................................................................
hsm: deleteImage - lock image by imgUUID and sdUUID hsm -> deleteImage: An image can reside on two storage domains concurrently (e.g. during LSM flow: between syncImage and deleteImage phases). Hence, deleting an image should take an exclusive lock on both image and storage domain IDs. I.e. since the image can exist on two storage domains concurrently, we cannot take both exclusive locks with the same name. Note: the modified lock in this patch is the only one uses merely an image UUID. Change-Id: I057b1f680276b1eb8a5d910260b7db4c4236fbf5 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1097648 Signed-off-by: Daniel Erez <[email protected]> Reviewed-on: http://gerrit.ovirt.org/28264 Reviewed-by: Nir Soffer <[email protected]> Reviewed-by: Federico Simoncelli <[email protected]> --- M vdsm/storage/hsm.py 1 file changed, 4 insertions(+), 1 deletion(-) Approvals: Nir Soffer: Looks good to me, but someone else must approve Federico Simoncelli: Looks good to me, approved Daniel Erez: Verified -- To view, visit http://gerrit.ovirt.org/28264 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I057b1f680276b1eb8a5d910260b7db4c4236fbf5 Gerrit-PatchSet: 3 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Daniel Erez <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Daniel Erez <[email protected]> Gerrit-Reviewer: Federico Simoncelli <[email protected]> Gerrit-Reviewer: Liron Aravot <[email protected]> Gerrit-Reviewer: Nir Soffer <[email protected]> Gerrit-Reviewer: [email protected] Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
