Gilad Chaplik has uploaded a new change for review.

Change subject: core: prevent persistence of runtime NUMA info
......................................................................

core: prevent persistence of runtime NUMA info

NUMA runtime info is shown wrongfully (bug 1100202),
also causes double pinning on the same node (bug fix).

This patch removes the persistence of NUMA runtime info, till
it gets resolved.

Change-Id: Ic87e09d1afae2e577f5a03ebaccc12a32ccf7681
Bug-Url: https://bugzilla.redhat.com/1147880
Signed-off-by: Gilad Chaplik <gchap...@redhat.com>
---
M 
backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VdsUpdateRunTimeInfo.java
1 file changed, 0 insertions(+), 13 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/81/35581/1

diff --git 
a/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VdsUpdateRunTimeInfo.java
 
b/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VdsUpdateRunTimeInfo.java
index 7a308b4..bd48239 100644
--- 
a/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VdsUpdateRunTimeInfo.java
+++ 
b/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VdsUpdateRunTimeInfo.java
@@ -206,19 +206,6 @@
         saveVmDevicesToDb();
         saveVmJobsToDb();
         saveVmGuestAgentNetworkDevices();
-        saveVmNumaNodeRuntimeData();
-    }
-
-    private void saveVmNumaNodeRuntimeData() {
-        if (!_vmStatisticsToSave.isEmpty()) {
-            final List<VmNumaNode> vmNumaNodesToUpdate = new ArrayList<>();
-            for(VmStatistics vmStats : _vmStatisticsToSave.values()) {
-                
vmNumaNodesToUpdate.addAll(vmStats.getvNumaNodeStatisticsList());
-            }
-            if (!vmNumaNodesToUpdate.isEmpty()) {
-                
getDbFacade().getVmNumaNodeDAO().massUpdateVmNumaNodeRuntimePinning(vmNumaNodesToUpdate);
-            }
-        }
     }
 
     private void saveVmGuestAgentNetworkDevices() {


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ic87e09d1afae2e577f5a03ebaccc12a32ccf7681
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