Change in ovirt-engine[ovirt-engine-3.6]: core: Handle exception if calling VDSM fails

tnisan at redhat.com tnisan at redhat.com
Wed Mar 2 10:05:48 UTC 2016


Tal Nisan has submitted this change and it was merged.

Change subject: core: Handle exception if calling VDSM fails
......................................................................


core: Handle exception if calling VDSM fails

When updating image data, we call VDSM to get image info.
If this call fails, the flow breaks and potentially leaving
database with incorrect data. This patch handles exception thrown
when calling VDSM.

Change-Id: Ia1aa477555cfb624724a5cc948120ea113f3f6ff
Bug-Url: https://bugzilla.redhat.com/1309294
Signed-off-by: Ala Hino <ahino at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RemoveSnapshotSingleDiskCommandBase.java
1 file changed, 14 insertions(+), 7 deletions(-)

Approvals:
  Tal Nisan: Passed CI tests
  Greg Padgett: Looks good to me, but someone else must approve
  Ala Hino: Verified
  Liron Aravot: Looks good to me, approved



-- 
To view, visit https://gerrit.ovirt.org/54135
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ia1aa477555cfb624724a5cc948120ea113f3f6ff
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.6
Gerrit-Owner: Ala Hino <ahino at redhat.com>
Gerrit-Reviewer: Ala Hino <ahino at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Greg Padgett <gpadgett at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Liron Aravot <laravot at redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>



More information about the Engine-commits mailing list