Noam Slomianko has uploaded a new change for review.

Change subject: webadmin: fix cluster creating in guide
......................................................................

webadmin: fix cluster creating in guide

When creating a cluster in the datacenter guide, policy units
configuretion was not saved.

added missing policy unit mapping from ClusterModel to VDSGroup

Change-Id: I7cc47558a06e4e0c000219ab2b8b34d9fac9f5a8
Bug-Url: https://bugzilla.redhat.com/1001182
Signed-off-by: Noam Slomianko <nslom...@redhat.com>
---
M 
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/datacenters/DataCenterGuideModel.java
1 file changed, 8 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/26/19826/1

diff --git 
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/datacenters/DataCenterGuideModel.java
 
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/datacenters/DataCenterGuideModel.java
index 502db78..f764a24 100644
--- 
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/datacenters/DataCenterGuideModel.java
+++ 
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/datacenters/DataCenterGuideModel.java
@@ -32,6 +32,7 @@
 import org.ovirt.engine.core.common.queries.SearchParameters;
 import org.ovirt.engine.core.common.queries.VdcQueryReturnValue;
 import org.ovirt.engine.core.common.queries.VdcQueryType;
+import org.ovirt.engine.core.common.scheduling.ClusterPolicy;
 import org.ovirt.engine.core.compat.Guid;
 import org.ovirt.engine.core.compat.StringHelper;
 import org.ovirt.engine.core.compat.Version;
@@ -59,6 +60,7 @@
 import org.ovirt.engine.ui.uicommonweb.models.storage.NfsStorageModel;
 import org.ovirt.engine.ui.uicommonweb.models.storage.SanStorageModel;
 import org.ovirt.engine.ui.uicommonweb.models.storage.StorageModel;
+import org.ovirt.engine.ui.uicommonweb.models.vms.key_value.KeyValueModel;
 import org.ovirt.engine.ui.uicommonweb.validation.IValidation;
 import org.ovirt.engine.ui.uicommonweb.validation.NotEmptyValidation;
 import org.ovirt.engine.ui.uicommonweb.validation.RegexValidation;
@@ -1387,6 +1389,12 @@
         cluster.setcompatibility_version(version);
         cluster.setVirtService((Boolean) 
model.getEnableOvirtService().getEntity());
         cluster.setGlusterService((Boolean) 
model.getEnableGlusterService().getEntity());
+        if (model.getClusterPolicy().getSelectedItem() != null) {
+            ClusterPolicy selectedPolicy = (ClusterPolicy) 
model.getClusterPolicy().getSelectedItem();
+            cluster.setClusterPolicyId(selectedPolicy.getId());
+            
cluster.setClusterPolicyProperties(KeyValueModel.convertProperties(model.getCustomPropertySheet()
+                    .getEntity()));
+        }
 
         model.startProgress(null);
 


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I7cc47558a06e4e0c000219ab2b8b34d9fac9f5a8
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.3
Gerrit-Owner: Noam Slomianko <nslom...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to