Itamar Heim 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:
Itamar Heim: Verified; Looks good to me, approved
--
To view, visit
http://gerrit.ovirt.org/21351
To unsubscribe, visit
http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I980e60ba5ad4fcc07752d065c3eaca03af025896
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.3
Gerrit-Owner: Federico Simoncelli <fsimonce(a)redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce(a)redhat.com>
Gerrit-Reviewer: Itamar Heim <iheim(a)redhat.com>
Gerrit-Reviewer: Sergey Gotliv <sgotliv(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server