Tal Nisan has submitted this change and it was merged. Change subject: core: Move getImageInfo call outside of transaction boundary ...................................................................... core: Move getImageInfo call outside of transaction boundary Currently, when syncing db records, we call vdsm to get volume info. This call is performed inside db transaction which could lead to failure regardless of the db performance. This could happen if storage is slow or network connection is slow. To fix this behavior, the call to vdsm moved outside of the transaction. This patch will simplify fixing the related bug. Change-Id: Ifac1ab925afd448d149fecc286d361a797c9d57b Bug-Url: https://bugzilla.redhat.com/1309294 Signed-off-by: Ala Hino <ahino@redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RemoveSnapshotSingleDiskCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RemoveSnapshotSingleDiskCommandBase.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RemoveSnapshotSingleDiskLiveCommand.java 3 files changed, 22 insertions(+), 14 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/54134 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ifac1ab925afd448d149fecc286d361a797c9d57b Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 Gerrit-Owner: Ala Hino <ahino@redhat.com> Gerrit-Reviewer: Ala Hino <ahino@redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini@redhat.com> Gerrit-Reviewer: Greg Padgett <gpadgett@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Liron Aravot <laravot@redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan@redhat.com> Gerrit-Reviewer: gerrit-hooks <automation@ovirt.org>