Change in vdsm[master]: migration: make stop() update internal status

Francesco Romani has uploaded a new change for review.

Change subject: migration: make stop() update internal status
......................................................................

migration: make stop() update internal status

The SourceThread.stop() operation should update
the internal status accordingly in case of success.

Previously, it was the caller that updated the status (!)
adding unneeded coupling.

Change-Id: I0ab50fc789dde969b2fb9ab969241ed4ad12545c
Signed-off-by: Francesco Romani <fromani(a)redhat.com&gt;
---
M vdsm/virt/migration.py
1 file changed, 4 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/20/40520/1

diff --git a/vdsm/virt/migration.py b/vdsm/virt/migration.py
index 1eaed05..61d5a7b 100644
--- a/vdsm/virt/migration.py
+++ b/vdsm/virt/migration.py
@@ -371,6 +371,10 @@
         except libvirt.libvirtError:
             if not self._preparingMigrationEvt:
                 raise
+        else:
+            self.status['status']['message'] = \
+                'Migration process cancelled'
+            return self.status
 
 
 def exponential_downtime(downtime, steps):


-- 
To view, visit https://gerrit.ovirt.org/40520
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I0ab50fc789dde969b2fb9ab969241ed4ad12545c
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com&gt;