Hello Arik Hadas,

I'd like you to do a code review.  Please visit

    http://gerrit.ovirt.org/31827

to review the following change.

Change subject: core: fix cpu profile validation on import VM
......................................................................

core: fix cpu profile validation on import VM

The cluster setting was missing on the VM object, thus we'll use the
destination cluster.

Change-Id: If228a9d2acd2db1a63bd9d12e798128dce6a3ee7
Signed-off-by: Arik Hadas <[email protected]>
---
M 
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ImportVmCommand.java
1 file changed, 1 insertion(+), 0 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/27/31827/1

diff --git 
a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ImportVmCommand.java
 
b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ImportVmCommand.java
index 92b2d8b..30f352a 100644
--- 
a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ImportVmCommand.java
+++ 
b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ImportVmCommand.java
@@ -1305,6 +1305,7 @@
     }
 
     protected boolean setAndValidateCpuProfile() {
+        getVm().getStaticData().setVdsGroupId(getVdsGroupId());
         return 
validate(CpuProfileHelper.setAndValidateCpuProfile(getVm().getStaticData(),
                 getVdsGroup().getcompatibility_version()));
     }


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: If228a9d2acd2db1a63bd9d12e798128dce6a3ee7
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.5
Gerrit-Owner: Gilad Chaplik <[email protected]>
Gerrit-Reviewer: Arik Hadas <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to