Michael Kublin has uploaded a new change for review.

Change subject: engine: Small code improvement at VdsEventListener.vdsUpEvent
......................................................................

engine: Small code improvement at VdsEventListener.vdsUpEvent

No reason to create a MultipleActionRunner when there are no vms

Change-Id: I66e1db82357a0b0cfc3260dc6aaf27ae51d3b803
Signed-off-by: Michael Kublin <[email protected]>
---
M 
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VdsEventListener.java
1 file changed, 7 insertions(+), 5 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/22/12022/1

diff --git 
a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VdsEventListener.java
 
b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VdsEventListener.java
index 394e48c..a84b3f9 100644
--- 
a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VdsEventListener.java
+++ 
b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VdsEventListener.java
@@ -158,11 +158,13 @@
                                         return parameters;
                                     }
                                 }));
-                        ExecutionContext executionContext = new 
ExecutionContext();
-                        executionContext.setMonitored(true);
-                        
Backend.getInstance().runInternalMultipleActions(VdcActionType.MigrateVmToServer,
-                                vmToServerParametersList,
-                                executionContext);
+                        if (vmToServerParametersList.size() > 0) {
+                            ExecutionContext executionContext = new 
ExecutionContext();
+                            executionContext.setMonitored(true);
+                            
Backend.getInstance().runInternalMultipleActions(VdcActionType.MigrateVmToServer,
+                                    vmToServerParametersList,
+                                    executionContext);
+                        }
 
                         // run dedicated vm logic
                         // not passing clientinfo will cause to launch on a VDS


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

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

Reply via email to