tp-qemu icon indicating copy to clipboard operation
tp-qemu copied to clipboard

qmp_basic: fix CommandNotFound

Open yiqianwei opened this issue 2 years ago • 6 comments

After qemu-kvm-5.1, migrate_set_speed is deprecated and replaced by migrate-set-parameters

ID: 2035434

Signed-off-by: Yiqian Wei [email protected]

yiqianwei avatar Jul 01 '22 09:07 yiqianwei

Auto results: (1/1) Host_RHEL.m8.u7.product_rhel.qcow2.virtio_scsi.up.virtio_net.Guest.RHEL.8.7.0.x86_64.io-github-autotest-qemu.qmp_basic.q35: PASS (29.17 s)

yiqianwei avatar Jul 01 '22 09:07 yiqianwei

@yanan-fu, Could you help to review this patch? Thanks

yiqianwei avatar Jul 01 '22 09:07 yiqianwei

In rhel9.1.0 host: (1/1) Host_RHEL.m9.u1.ovmf.qcow2.virtio_scsi.up.virtio_net.Guest.RHEL.9.1.0.x86_64.io-github-autotest-qemu.qmp_basic.q35: PASS (51.38 s)

In rhel8.7.0 host: (1/1) Host_RHEL.m8.u7.product_rhel.ovmf.qcow2.virtio_scsi.up.virtio_net.Guest.RHEL.8.7.0.x86_64.io-github-autotest-qemu.qmp_basic.q35: PASS (50.20 s)

@yanan-fu , I have updated it, please help to review this patch again ? Thanks

yiqianwei avatar Jul 06 '22 06:07 yiqianwei

Looks good on my side Acked-by: [email protected]

jingzhao84 avatar Jul 21 '22 09:07 jingzhao84

I already commented before, please check: https://github.com/autotest/tp-qemu/pull/3383#discussion_r917654590

yanan-fu avatar Jul 27 '22 03:07 yanan-fu

(1/1) Host_RHEL.m9.u2.qcow2.virtio_scsi.up.virtio_net.Guest.RHEL.9.2.0.x86_64.io-github-autotest-qemu.qmp_basic: PASS (17.74 s)

yiqianwei avatar Nov 30 '22 09:11 yiqianwei

@yanan-fu , This patch has been a long time, please help review it. thanks a lot

yiqianwei avatar Mar 06 '23 06:03 yiqianwei

@yiqianwei Could you please split into 2 commits, 1 for the update with migrate part, and another one for the change with check_error_resp.

yanan-fu avatar Mar 15 '23 01:03 yanan-fu

test results: (1/1) Host_RHEL.m9.u2.ovmf.qcow2.virtio_scsi.up.virtio_net.Guest.RHEL.9.2.0.x86_64.io-github-autotest-qemu.qmp_basic.q35: PASS (21.23 s

yiqianwei avatar Mar 16 '23 09:03 yiqianwei

@yiqianwei Could you please split into 2 commits, 1 for the update with migrate part, and another one for the change with check_error_resp.

@yanan-fu , I have updated it, please review this patch. thanks

yiqianwei avatar Mar 17 '23 06:03 yiqianwei