Re: [Qemu-devel] [PATCH 1/3] tests/acceptance/migration: fix post migration check

2019-09-17 Thread Dr. David Alan Gilbert
* Balamuruhan S (bal...@linux.ibm.com) wrote: > On Mon, Sep 16, 2019 at 07:50:44PM +0100, Dr. David Alan Gilbert wrote: > > * Balamuruhan S (bal...@linux.ibm.com) wrote: > > > assert `query-migrate` in target doesn't give migration > > > status and test errors even if migration succeeds. > > > > >

Re: [Qemu-devel] [PATCH 1/3] tests/acceptance/migration: fix post migration check

2019-09-17 Thread Balamuruhan S
On Mon, Sep 16, 2019 at 07:50:44PM +0100, Dr. David Alan Gilbert wrote: > * Balamuruhan S (bal...@linux.ibm.com) wrote: > > assert `query-migrate` in target doesn't give migration > > status and test errors even if migration succeeds. > > > > In target: > > {'execute': 'query-migrate'} > > {"retur

Re: [Qemu-devel] [PATCH 1/3] tests/acceptance/migration: fix post migration check

2019-09-16 Thread Dr. David Alan Gilbert
* Balamuruhan S (bal...@linux.ibm.com) wrote: > assert `query-migrate` in target doesn't give migration > status and test errors even if migration succeeds. > > In target: > {'execute': 'query-migrate'} > {"return": {}} On which version of qemu? On the current version I see: {"QMP": {"version":

[Qemu-devel] [PATCH 1/3] tests/acceptance/migration: fix post migration check

2019-09-16 Thread Balamuruhan S
assert `query-migrate` in target doesn't give migration status and test errors even if migration succeeds. In target: {'execute': 'query-migrate'} {"return": {}} Signed-off-by: Balamuruhan S --- tests/acceptance/migration.py | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --g

[Qemu-devel] [PATCH 1/3] tests/acceptance/migration: fix post migration check

2019-08-05 Thread Balamuruhan S
assert `query-migrate` in target doesn't give migration status and test errors even if migration succeeds. In target: {'execute': 'query-migrate'} {"return": {}} Signed-off-by: Balamuruhan S --- tests/acceptance/migration.py | 1 - 1 file changed, 1 deletion(-) diff --git a/tests/acceptance/mi