Change in ovirt-engine[master]: core: cleanup in destroy vm

ahadas at redhat.com ahadas at redhat.com
Mon Aug 22 13:58:39 UTC 2016


Arik Hadas has submitted this change and it was merged.

Change subject: core: cleanup in destroy vm
......................................................................


core: cleanup in destroy vm

1. query vm dynamic data only if the vm is actually destroyed
2. rename curVm to vm (no other VM is defined)
3. remove unused argument from #changeStatus

Change-Id: I4a168284a366b19b05ab2df1df3869f589eec78b
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, 6 insertions(+), 11 deletions(-)

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



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I4a168284a366b19b05ab2df1df3869f589eec78b
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>



More information about the Engine-commits mailing list