Allon Mureinik has submitted this change and it was merged.
Change subject: backend: unlock images when GetImageInfo fails
......................................................................
backend: unlock images when GetImageInfo fails
In BaseImagesCommand.endSuccessfully() we try to update the image
information contacting VDSM. If VDSM is down or for any other error
we should fail gracefully (logging) and release the locks on the
image.
Bug-Url:
https://bugzilla.redhat.com/show_bug.cgi?id=1018876
Change-Id: I980e60ba5ad4fcc07752d065c3eaca03af025896
Signed-off-by: Federico Simoncelli <fsimonce(a)redhat.com>
---
M
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/BaseImagesCommand.java
1 file changed, 17 insertions(+), 10 deletions(-)
Approvals:
Allon Mureinik: Verified; Looks good to me, approved
--
To view, visit
http://gerrit.ovirt.org/20669
To unsubscribe, visit
http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I980e60ba5ad4fcc07752d065c3eaca03af025896
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Federico Simoncelli <fsimonce(a)redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com>
Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce(a)redhat.com>
Gerrit-Reviewer: Liron Ar <laravot(a)redhat.com>
Gerrit-Reviewer: Sergey Gotliv <sgotliv(a)redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server