Gilad Chaplik has uploaded a new change for review.

Change subject: webadmin: cpu pinning is disabled for Local SD
......................................................................

webadmin: cpu pinning is disabled for Local SD

Should be enabled without any consideration to other VM's migration properties.

Change-Id: I4706419295d67f7a022a85bc61f75bb46def2b6b
Signed-off-by: Gilad Chaplik <[email protected]>
---
M 
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/VmModelBehaviorBase.java
1 file changed, 5 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/91/20591/1

diff --git 
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/VmModelBehaviorBase.java
 
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/VmModelBehaviorBase.java
index 69cf045..7b13fa3 100644
--- 
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/VmModelBehaviorBase.java
+++ 
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/VmModelBehaviorBase.java
@@ -691,7 +691,11 @@
         if (getModel().getSelectedCluster() != null) {
             VDSGroup cluster = getModel().getSelectedCluster();
             String compatibilityVersion = 
cluster.getcompatibility_version().toString();
-            boolean hasCpuPinning = 
Boolean.FALSE.equals(getModel().getIsAutoAssign().getEntity());
+            boolean isLocalSD = getModel().getSelectedDataCenter() != null
+                    && 
StorageType.LOCALFS.equals(getModel().getSelectedDataCenter().getStorageType());
+
+            // cpu pinning is available on Local SD with no consideration for 
auto assign value
+            boolean hasCpuPinning = 
Boolean.FALSE.equals(getModel().getIsAutoAssign().getEntity()) || isLocalSD;
 
             if 
(Boolean.FALSE.equals(AsyncDataProvider.getConfigValuePreConverted(ConfigurationValues.CpuPinningEnabled,
                     compatibilityVersion))) {


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

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

Reply via email to