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