Change in ovirt-engine[master]: core: change import vm to be VmCommand - part 2

ahadas at redhat.com ahadas at redhat.com
Tue Nov 25 16:10:52 UTC 2014


Arik Hadas has submitted this change and it was merged.

Change subject: core: change import vm to be VmCommand - part 2
......................................................................


core: change import vm to be VmCommand - part 2

Inline checkStoragePool method call and moved fillMacAddress method, to
reduce the coupling between ImportVmCommand and
MoveOrCopyTemplateCommand.

Change-Id: Ie7063d22fbe4bd81de24f050660e35b3ea7b96cd
Signed-off-by: Arik Hadas <ahadas at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ImportVmCommand.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/MoveOrCopyTemplateCommand.java
2 files changed, 9 insertions(+), 9 deletions(-)

Approvals:
  Omer Frenkel: Looks good to me, approved
  Arik Hadas: Verified



-- 
To view, visit http://gerrit.ovirt.org/34986
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ie7063d22fbe4bd81de24f050660e35b3ea7b96cd
Gerrit-PatchSet: 6
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Liron Aravot <laravot at redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel at redhat.com>
Gerrit-Reviewer: Vered Volansky <vvolansk at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list