Change in ovirt-engine[master]: core: handle down VM in separate thread on destroy

ahadas at redhat.com ahadas at redhat.com
Tue Jun 17 10:49:24 UTC 2014


Arik Hadas has submitted this change and it was merged.

Change subject: core: handle down VM in separate thread on destroy
......................................................................


core: handle down VM in separate thread on destroy

We used to execute the handling of down VM in a new transaction in
DestroyVmVDSCommand so it won't be affected by the current transaction.
Now that this handling is made on a different thread, there is no need
for the new transaction anymore.

Change-Id: I1f20edfece133465aade00127ff973da9b726975
Signed-off-by: Arik Hadas <ahadas at redhat.com>
---
M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/DestroyVmVDSCommand.java
1 file changed, 2 insertions(+), 20 deletions(-)

Approvals:
  Omer Frenkel: Looks good to me, approved
  Arik Hadas: Verified



-- 
To view, visit http://gerrit.ovirt.org/28222
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I1f20edfece133465aade00127ff973da9b726975
Gerrit-PatchSet: 6
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: Omer Frenkel <ofrenkel at redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourfali at redhat.com>
Gerrit-Reviewer: Roy Golan <rgolan at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list