Daniel Erez has uploaded a new change for review.

Change subject: core: LiveMigrateDisk - check isRunning VMStatus
......................................................................

core: LiveMigrateDisk - check isRunning VMStatus

LiveMigrateDiskCommand - check isRunning VMStatus instead of
isUpOrPaused [to prevent command failure when the VM changes
running status (e.g. RebootInProgress) while live migrating)].

Change-Id: I546dbe5e225ee0db5ca190d3237b70c5a425e004
Bug-Url: https://bugzilla.redhat.com/923992
Signed-off-by: Daniel Erez <de...@redhat.com>
---
M 
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/lsm/LiveMigrateDiskCommand.java
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/42/13242/1

diff --git 
a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/lsm/LiveMigrateDiskCommand.java
 
b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/lsm/LiveMigrateDiskCommand.java
index d443ec6..182ba47 100644
--- 
a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/lsm/LiveMigrateDiskCommand.java
+++ 
b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/lsm/LiveMigrateDiskCommand.java
@@ -35,7 +35,7 @@
         getParameters().setDiskAlias(getDiskAlias());
         getParameters().setImageGroupID(getImageGroupId());
         getParameters().setCommandType(getActionType());
-        
getParameters().setTaskGroupSuccess(getParameters().getTaskGroupSuccess() && 
getVm().getStatus().isUpOrPaused());
+        
getParameters().setTaskGroupSuccess(getParameters().getTaskGroupSuccess() && 
getVm().getStatus().isRunning());
     }
 
     /* Overridden CommandBase Methods */


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

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

Reply via email to