Thomas Huth 8f023a0bd9 tests/qtest/migration-test: Fix the check for a successful run of analyze-migration.py ... If analyze-migration.py cannot be run or crashes, the error is currently ignored since the code only checks for nonzero values in case the child exited properly. For example, if you run the test with a non-existing Python interpreter, it still succeeds: $ PYTHON=wrongpython QTEST_QEMU_BINARY=./qemu-system-x86_64 tests/qtest/migration-test ... # Running /x86_64/migration/analyze-script # Using machine type: pc-q35-9.1 # starting QEMU: exec ./qemu-system-x86_64 -qtest unix:/tmp/qtest-417639.sock -qtest-log /dev/null -chardev socket,path=/tmp/qtest-417639.qmp,id=char0 -mon chardev=char0,mode=control -display none -audio none -accel kvm -accel tcg -machine pc-q35-9.1, -name source,debug-threads=on -m 150M -serial file:/tmp/migration-test-XPLUN2/src_serial -drive if=none,id=d0,file=/tmp/migration-test-XPLUN2/bootsect,format=raw -device ide-hd,drive=d0,secs=1,cyls=1,heads=1 -uuid 11111111-1111-1111-1111-111111111111 -accel qtest # starting QEMU: exec ./qemu-system-x86_64 -qtest unix:/tmp/qtest-417639.sock -qtest-log /dev/null -chardev socket,path=/tmp/qtest-417639.qmp,id=char0 -mon chardev=char0,mode=control -display none -audio none -accel kvm -accel tcg -machine pc-q35-9.1, -name target,debug-threads=on -m 150M -serial file:/tmp/migration-test-XPLUN2/dest_serial -incoming tcp:127.0.0.1:0 -drive if=none,id=d0,file=/tmp/migration-test-XPLUN2/bootsect,format=raw -device ide-hd,drive=d0,secs=1,cyls=1,heads=1 -accel qtest ** ERROR:../../devel/qemu/tests/qtest/migration-test.c:1603:test_analyze_script: code should not be reached migration-test: ../../devel/qemu/tests/qtest/libqtest.c:240: qtest_wait_qemu: Assertion `pid == s->qemu_pid' failed. migration-test: ../../devel/qemu/tests/qtest/libqtest.c:240: qtest_wait_qemu: Assertion `pid == s->qemu_pid' failed. ok 2 /x86_64/migration/analyze-script ... Let's better fail the test in case the child did not exit properly, too. Signed-off-by: Thomas Huth <thuth@redhat.com> Reviewed-by: Fabiano Rosas <farosas@suse.de> Reviewed-by: Peter Xu <peterx@redhat.com> Signed-off-by: Fabiano Rosas <farosas@suse.de>		2024-05-22 17:34:41 -03:00
..
avocado	tests/avocado: update sunxi kernel from armbian to 6.6.16	2024-04-30 15:06:26 +01:00
bench	tests/bench: Add bufferiszero-bench	2024-05-03 08:03:35 -07:00
data
decode
docker	tests/docker/dockerfiles: Update container files with "lcitool-refresh"	2024-05-17 11:16:43 +02:00
fp
guest-debug	tests: Gently exit from GDB when tests complete	2024-05-17 16:49:04 +02:00
image-fuzzer
include
keys
lcitool	tests/lcitool/projects/qemu.yml: Sort entries alphabetically again	2024-05-17 11:16:37 +02:00
migration
multiboot
perf/block/qcow2
plugin	tests/plugin/inline: add test for conditional callback	2024-05-16 08:55:23 +01:00
qapi-schema	qapi: Rename visitor parameter @variants to @alternatives	2024-05-06 12:38:27 +02:00
qemu-iotests	migration: Remove 'blk/-b' option from migrate commands	2024-05-08 09:20:58 -03:00
qtest	tests/qtest/migration-test: Fix the check for a successful run of analyze-migration.py	2024-05-22 17:34:41 -03:00
rocker
tcg	tests/tcg: don't append QEMU_OPTS for armv6m-undef test	2024-05-16 08:47:01 +01:00
tsan
uefi-test-tools
unit	tests: add testing of parameter=1 for SMP topology	2024-05-17 16:49:04 +02:00
vm	tests: Update our CI to use CentOS Stream 9 instead of 8	2024-04-25 07:03:04 +02:00
vmstate-static-checker-data
dbus-vmstate-daemon.sh
Makefile.include
meson.build	tests: only build plugins if TCG is enabled	2024-04-18 11:17:27 +02:00
test-qht-par.c
vhost-user-bridge.c