Omer Frenkel has submitted this change and it was merged.
Change subject: core: remove MigrateVmCommand#canMigrateVm
......................................................................
core: remove MigrateVmCommand#canMigrateVm
All that MigrateVmCommand#canDoAction was doing was to delegate to
canMigrateVm method and canMigrateVm method was not called from anywhere
else. Thus, this patch removes the canMigrateVm method and put its body
in the canDoAction method instead.
InternalMigrateVmCommand is also changed accordingly.
Change-Id: If88012b8bdbc7d23ad9350acf207431727e1d51e
Signed-off-by: Arik Hadas <ahadas(a)redhat.com>
---
M
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/InternalMigrateVmCommand.java
M
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/MigrateVmCommand.java
2 files changed, 11 insertions(+), 13 deletions(-)
Approvals:
Omer Frenkel: Verified; Looks good to me, approved
--
To view, visit
http://gerrit.ovirt.org/18468
To unsubscribe, visit
http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: If88012b8bdbc7d23ad9350acf207431727e1d51e
Gerrit-PatchSet: 5
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Arik Hadas <ahadas(a)redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server