Change in ovirt-engine[ovirt-engine-4.1]: core: always clone on import from cfme

Code Review gerrit at ovirt.org
Wed Aug 9 06:46:32 UTC 2017


>From Oved Ourfali <oourfali at redhat.com>:

Oved Ourfali has submitted this change and it was merged. ( https://gerrit.ovirt.org/80416 )

Change subject: core: always clone on import from cfme
......................................................................


core: always clone on import from cfme

When importing a VM not as a new entity (i.e., not cloning it), we preserve
its MAC address. This is also the case  when the mac address is out of the
MACs pool's range, which leads to unplugged NIC(s) for the imported VM.
As a temporary workaround, we change the import from external URL (i.e.,
only from external provider by CFME at the moment) to always clone the VM,
also when the VM doesn't exist in the system. This way, the MAC addresses
are reallocated during the import process.

Change-Id: I709a0dee90eeade0f7715e3be3242a6ce15e1f65
Bug-Url: https://bugzilla.redhat.com/1437152
Signed-off-by: Arik Hadas <ahadas at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/exportimport/ImportVmFromExternalUrlCommand.java
1 file changed, 4 insertions(+), 2 deletions(-)

Approvals:
  Tomas Jelinek: Looks good to me, approved
  Jenkins CI: Passed CI tests
  Arik Hadas: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I709a0dee90eeade0f7715e3be3242a6ce15e1f65
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-4.1
Gerrit-Owner: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Oved Ourfali <oourfali at redhat.com>
Gerrit-Reviewer: Tomas Jelinek <tjelinek at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>


More information about the Engine-commits mailing list