Change in ovirt-engine[master]: core: simplify hibernate vm flow

ahadas at redhat.com ahadas at redhat.com
Mon Sep 26 12:39:55 UTC 2016


Arik Hadas has submitted this change and it was merged.

Change subject: core: simplify hibernate vm flow
......................................................................


core: simplify hibernate vm flow

Remove redundant transaction when updating the status of the VM. It is
not needed since HibernateVDSCommand extends ManagingVmCommand that
locks the VM, thus prevents other updates at the same time.

Change-Id: Ia0e0351a8b43413a6e06f89fee0b8d9d13ba364d
Signed-off-by: Arik Hadas <ahadas at redhat.com>
---
M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/HibernateVDSCommand.java
M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VmManager.java
2 files changed, 23 insertions(+), 22 deletions(-)

Approvals:
  Shahar Havivi: Looks good to me, approved
  Jenkins CI: Passed CI tests
  Arik Hadas: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ia0e0351a8b43413a6e06f89fee0b8d9d13ba364d
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Shahar Havivi <shavivi at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>



More information about the Engine-commits mailing list