Moti Asayag has uploaded a new change for review.

Change subject: engine: Use proper method to mark net changes as saved
......................................................................

engine: Use proper method to mark net changes as saved

The patch reduces the need for updating the entire VdsDynamic
data by using a designated method for saving the dirty-network
indicator of the host.

Change-Id: Ib4c54214a275a1653c46bca7c95ff847975dd7d4
Signed-off-by: Moti Asayag <masa...@redhat.com>
---
M 
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/host/CommitNetworkChangesCommand.java
1 file changed, 3 insertions(+), 7 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/05/13905/1

diff --git 
a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/host/CommitNetworkChangesCommand.java
 
b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/host/CommitNetworkChangesCommand.java
index 09150c2..36cf305 100644
--- 
a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/host/CommitNetworkChangesCommand.java
+++ 
b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/host/CommitNetworkChangesCommand.java
@@ -1,6 +1,5 @@
 package org.ovirt.engine.core.bll.network.host;
 
-import org.ovirt.engine.core.bll.Backend;
 import org.ovirt.engine.core.bll.VdsCommand;
 import org.ovirt.engine.core.common.AuditLogType;
 import org.ovirt.engine.core.common.action.VdsActionParameters;
@@ -16,14 +15,11 @@
 
     @Override
     protected void executeCommand() {
-        VDSReturnValue retVal = Backend
-                .getInstance()
-                .getResourceManager()
-                .RunVdsCommand(VDSCommandType.SetSafeNetworkConfig,
+        VDSReturnValue retVal =
+                runVdsCommand(VDSCommandType.SetSafeNetworkConfig,
                         new 
VdsIdVDSCommandParametersBase(getParameters().getVdsId()));
 
-        getVds().setNetConfigDirty(false);
-        getDbFacade().getVdsDynamicDao().update(getVds().getDynamicData());
+        
getDbFacade().getVdsDynamicDao().updateNetConfigDirty(getParameters().getVdsId(),
 false);
         setSucceeded(retVal.getSucceeded());
     }
 


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ib4c54214a275a1653c46bca7c95ff847975dd7d4
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Moti Asayag <masa...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to