Change in ovirt-engine[master]: core: remove redundant things from destroy vm

[ahadas at redhat.com]

Arik Hadas has submitted this change and it was merged.

Change subject: core: remove redundant things from destroy vm
......................................................................


core: remove redundant things from destroy vm

Clear the assigned numa nodes in ProcessDownVm instead of in
DestroyVm. The reason is that destroy VM is an asynchronous
operation and the numa nodes should be cleared only when it
succeeds.

Change-Id: I5b124cdf39f97f7c91a6c53e20eaa44f1ae61ce1
Signed-off-by: Arik Hadas <ahadas at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ProcessDownVmCommand.java
M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/DestroyVmVDSCommand.java
2 files changed, 14 insertions(+), 15 deletions(-)

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



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I5b124cdf39f97f7c91a6c53e20eaa44f1ae61ce1
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: Jenkins CI
Gerrit-Reviewer: Shahar Havivi <shavivi at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>