Dudi Maroshi has uploaded a new change for review.

Change subject: engine:Numa nodes statistic clear dynamic data info
......................................................................

engine:Numa nodes statistic clear dynamic data info

When storing Numa nodes statistics. The Numa node data containts only the 
statistics.
Therefore when storing the Numa node it clear any other dynamic information
The fix is update the statistics into the current stored Numa node.

Change-Id: I3b13356a50e594148fe6fe08c341b0a14e82b234
Bug-Url: https://bugzilla.redhat.com/1177154
Signed-off-by: Dudi Maroshi <[email protected]>
---
M 
backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/HostMonitoring.java
1 file changed, 4 insertions(+), 3 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/14/37614/1

diff --git 
a/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/HostMonitoring.java
 
b/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/HostMonitoring.java
index f0673c5..6ef8394 100644
--- 
a/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/HostMonitoring.java
+++ 
b/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/HostMonitoring.java
@@ -258,10 +258,11 @@
                 nodesMap.put(node.getIndex(), node);
             }
             for (VdsNumaNode node : updateNumaNodes) {
-                if (nodesMap.containsKey(node.getIndex())) {
-                    node.setId(nodesMap.get(node.getIndex()).getId());
+                VdsNumaNode dbNode = nodesMap.get(node.getIndex());
+                if (dbNode != null) {
                     if (node.getNumaNodeStatistics() != null) {
-                        vdsNumaNodesToSave.add(node);
+                        
dbNode.setNumaNodeStatistics(node.getNumaNodeStatistics());
+                        vdsNumaNodesToSave.add(dbNode);
                     }
                 }
             }


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

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

Reply via email to