Change in vdsm[master]: vm: use helper instead of reimplementing it

fromani at redhat.com fromani at redhat.com
Wed Feb 19 14:09:08 UTC 2014


Francesco Romani has uploaded a new change for review.

Change subject: vm: use helper instead of reimplementing it
......................................................................

vm: use helper instead of reimplementing it

use the _incomingMigrationPending() to detect
if we need to wait for incoming migrations instead
to check the status direcly.

Change-Id: I3249b131990f46700eccb46448589b17be0b5a3b
Signed-off-by: Francesco Romani <fromani at redhat.com>
---
M vdsm/vm.py
1 file changed, 1 insertion(+), 2 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/46/24746/1

diff --git a/vdsm/vm.py b/vdsm/vm.py
index e2e0f47..f7e95e2 100644
--- a/vdsm/vm.py
+++ b/vdsm/vm.py
@@ -2303,8 +2303,7 @@
                 self._ongoingCreations.release()
                 self.log.debug("_ongoingCreations released")
 
-            if ('migrationDest' in self.conf or 'restoreState' in self.conf) \
-                    and self.lastStatus != 'Down':
+            if self._incomingMigrationPending() and self.lastStatus != 'Down':
                 self._waitForIncomingMigrationFinish()
 
             self.lastStatus = 'Up'


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I3249b131990f46700eccb46448589b17be0b5a3b
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani at redhat.com>


More information about the vdsm-patches mailing list