Change in ovirt-engine[master]: backend: unlock images when GetImageInfo fails

[amureini at redhat.com]

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 at 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 at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Liron Ar <laravot at redhat.com>
Gerrit-Reviewer: Sergey Gotliv <sgotliv at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server