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@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@redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Shahar Havivi <shavivi@redhat.com> Gerrit-Reviewer: gerrit-hooks <automation@ovirt.org>