Bug 681280 - [vdsm] when qemu process dies during migration vdsm reports the vm status as up*(not-responding)
Summary: [vdsm] when qemu process dies during migration vdsm reports the vm status as ...
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat Enterprise Linux 6
Classification: Red Hat
Component: vdsm
Version: 6.1
Hardware: Unspecified
OS: Unspecified
medium
low
Target Milestone: rc
: ---
Assignee: Dan Kenigsberg
QA Contact: yeylon@redhat.com
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2011-03-01 16:30 UTC by David Naori
Modified: 2016-04-18 06:38 UTC (History)
10 users (show)

Fixed In Version: vdsm-4.9-52.el6
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2011-08-19 15:12:05 UTC
Target Upstream Version:

Attachments (Terms of Use)
full vdsm log (10.65 MB, application/octet-stream)
2011-03-01 16:30 UTC, David Naori 		no flags Details
View All Add an attachment (proposed patch, testcase, etc.)

Description David Naori 2011-03-01 16:30:58 UTC 
Created attachment 481676 [details]
full vdsm log

Description of problem: 
when migrating multiple vms and qemu process dies on the source- vdsm reports the vm status as up*(not-responding) while he is actually down.

Version-Release number of selected component (if applicable):

libvirt-0.8.7-8
vdsm-4.9-51

How reproducible:
always


Steps to Reproduce:
1.migrate 5 vms, 
2.when migration start kill qemu prosess in the source
  
full vdsm log attached.
Comment 2 Dan Kenigsberg 2011-03-02 07:53:57 UTC 
Did my patch solve the problem?

diff --git a/vdsm/vm.py b/vdsm/vm.py
index 24ea8a7..a864f69 100644
--- a/vdsm/vm.py
+++ b/vdsm/vm.py
@@ -337,7 +337,7 @@ class MigrationSourceThread(threading.Thread):
         if self._mode == 'file' or self._method != 'online':
             self._vm.cont()
         # either way, migration has finished
-        self._vm._lastStatus = 'Up'
+        self._vm.lastStatus = 'Up'
 
     def _waitForOutgoingMigration(self):
         now = time.time()
Comment 3 David Naori 2011-03-02 08:24:19 UTC 
I have tested your patch and it works.
Comment 4 David Naori 2011-03-08 13:54:27 UTC 
Verified on vdsm-4.9-52.el6
Note You need to log in before you can comment on or make changes to this bug.