Gilad Chaplik has uploaded a new change for review.

Change subject: core: fix numa update info
......................................................................

core: fix numa update info

* Adding missing numa info to vds.clone()
* Update numa only when it's reported

Change-Id: I13c587521d9233b74a27aba59a5c1c7ca386e1db
Bug-Url: https://bugzilla.redhat.com/1069303
Signed-off-by: Gilad Chaplik <gchap...@redhat.com>
---
M 
backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VDS.java
M 
backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VdsManager.java
2 files changed, 6 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/92/33192/1

diff --git 
a/backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VDS.java
 
b/backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VDS.java
index 80c2d1f..00d9fb6 100644
--- 
a/backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VDS.java
+++ 
b/backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VDS.java
@@ -182,6 +182,8 @@
         vds.setHighlyAvailableIsActive(getHighlyAvailableIsActive());
         
vds.setHighlyAvailableGlobalMaintenance(getHighlyAvailableGlobalMaintenance());
         
vds.setHighlyAvailableLocalMaintenance(getHighlyAvailableLocalMaintenance());
+        vds.setNumaNodeList(getNumaNodeList());
+        vds.setAutoNumaBalancing(getAutoNumaBalancing());
 
         return vds;
     }
diff --git 
a/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VdsManager.java
 
b/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VdsManager.java
index 750a029..3a8a43a 100644
--- 
a/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VdsManager.java
+++ 
b/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VdsManager.java
@@ -391,6 +391,10 @@
      * @param vds
      */
     public void updateNumaData(final VDS vds) {
+        if (vds.getNumaNodeList() == null || vds.getNumaNodeList().isEmpty()) {
+            return;
+        }
+
         final List<VdsNumaNode> numaNodesToSave = new ArrayList<>();
         final List<VdsNumaNode> numaNodesToUpdate = new ArrayList<>();
         final List<Guid> numaNodesToRemove = new ArrayList<>();


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I13c587521d9233b74a27aba59a5c1c7ca386e1db
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.5
Gerrit-Owner: Gilad Chaplik <gchap...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to