Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

stop_con_vm_during_virtiofs: add a new testcase #3864

Merged
merged 2 commits into from
Nov 29, 2023
Merged

stop_con_vm_during_virtiofs: add a new testcase #3864

merged 2 commits into from
Nov 29, 2023

Conversation

nickzhq
Copy link
Contributor

@nickzhq nickzhq commented Oct 31, 2023 •
edited
Loading

id: 1311

@nickzhq nickzhq force-pushed the New_stop_con_vm_during_virtiofs_2223243 branch 5 times, most recently from 4c846f6 to 929d7cf Compare October 31, 2023 06:28
@nickzhq
Copy link
Contributor Author

nickzhq commented Oct 31, 2023

Linux:
(1/1) Host_RHEL.m9.u3.ovmf.qcow2.virtio_scsi.up.virtio_net.Guest.RHEL.9.0.0.x86_64.io-github-autotest-qemu.stop_cont_vm_during_virtiofs.q35: STARTED
(1/1) Host_RHEL.m9.u3.ovmf.qcow2.virtio_scsi.up.virtio_net.Guest.RHEL.9.0.0.x86_64.io-github-autotest-qemu.stop_cont_vm_during_virtiofs.q35: PASS (157.11 s)
RESULTS : PASS 1 | ERROR 0 | FAIL 0 | SKIP 0 | WARN 0 | INTERRUPT 0 | CANCEL 0

@nickzhq nickzhq force-pushed the New_stop_con_vm_during_virtiofs_2223243 branch 5 times, most recently from 1a74d15 to dd9e870 Compare November 2, 2023 04:12
@nickzhq
Copy link
Contributor Author

nickzhq commented Nov 2, 2023

(1/1) Host_RHEL.m9.u3.ovmf.qcow2.virtio_scsi.up.virtio_net.Guest.RHEL.9.0.0.x86_64.io-github-autotest-qemu.stop_cont_vm_during_virtiofs.q35: STARTED
(1/1) Host_RHEL.m9.u3.ovmf.qcow2.virtio_scsi.up.virtio_net.Guest.RHEL.9.0.0.x86_64.io-github-autotest-qemu.stop_cont_vm_during_virtiofs.q35: PASS (163.07 s)

(1/1) Host_RHEL.m9.u3.ovmf.qcow2.virtio_scsi.up.virtio_net.Guest.Win10.x86_64.io-github-autotest-qemu.stop_cont_vm_during_virtiofs.q35: STARTED
(1/1) Host_RHEL.m9.u3.ovmf.qcow2.virtio_scsi.up.virtio_net.Guest.Win10.x86_64.io-github-autotest-qemu.stop_cont_vm_during_virtiofs.q35: PASS (215.43 s)

@nickzhq nickzhq force-pushed the New_stop_con_vm_during_virtiofs_2223243 branch from dd9e870 to 5d11b95 Compare November 2, 2023 05:15
@nickzhq
Copy link
Contributor Author

nickzhq commented Nov 2, 2023

@xiagao @zhencliu @YongxueHong Please take a review, thanks!

xiagao
xiagao reviewed Nov 2, 2023
View reviewed changes
qemu/tests/cfg/stop_cont_vm_during_virtiofs.cfg Outdated Show resolved Hide resolved
qemu/tests/cfg/stop_cont_vm_during_virtiofs.cfg Outdated Show resolved Hide resolved
qemu/tests/stop_cont_vm_during_virtiofs.py Outdated Show resolved Hide resolved
qemu/tests/stop_cont_vm_during_virtiofs.py Outdated Show resolved Hide resolved
qemu/tests/stop_cont_vm_during_virtiofs.py Outdated Show resolved Hide resolved
qemu/tests/stop_cont_vm_during_virtiofs.py Outdated Show resolved Hide resolved
qemu/tests/stop_cont_vm_during_virtiofs.py Outdated Show resolved Hide resolved
@nickzhq nickzhq force-pushed the New_stop_con_vm_during_virtiofs_2223243 branch 4 times, most recently from 33762fa to 7f8a222 Compare November 3, 2023 02:48
@nickzhq nickzhq force-pushed the New_stop_con_vm_during_virtiofs_2223243 branch from 7f8a222 to accc5f2 Compare November 16, 2023 05:17
@nickzhq
Copy link
Contributor Author

nickzhq commented Nov 16, 2023

@xiagao @YongxueHong @zhencliu Please take a review, thanks!

@nickzhq nickzhq force-pushed the New_stop_con_vm_during_virtiofs_2223243 branch from accc5f2 to 879a5bc Compare November 16, 2023 06:20
@nickzhq
Copy link
Contributor Author

nickzhq commented Nov 16, 2023

windows:
(1/1) Host_RHEL.m9.u3.ovmf.qcow2.virtio_scsi.up.virtio_net.Guest.Win10.x86_64.io-github-autotest-qemu.virtio_fs_subtest_during_io.q35: STARTED
(1/1) Host_RHEL.m9.u3.ovmf.qcow2.virtio_scsi.up.virtio_net.Guest.Win10.x86_64.io-github-autotest-qemu.virtio_fs_subtest_during_io.q35: PASS (265.93 s)

linux
(1/1) Host_RHEL.m9.u3.ovmf.qcow2.virtio_scsi.up.virtio_net.Guest.RHEL.9.0.0.x86_64.io-github-autotest-qemu.virtio_fs_subtest_during_io.q35: STARTED
(1/1) Host_RHEL.m9.u3.ovmf.qcow2.virtio_scsi.up.virtio_net.Guest.RHEL.9.0.0.x86_64.io-github-autotest-qemu.virtio_fs_subtest_during_io.q35: PASS (91.11 s)

xiagao
xiagao reviewed Nov 17, 2023
View reviewed changes
qemu/tests/virtio_fs_subtest_during_io.py Outdated Show resolved Hide resolved
qemu/tests/virtio_fs_subtest_during_io.py Outdated Show resolved Hide resolved
qemu/tests/virtio_fs_subtest_during_io.py Outdated Show resolved Hide resolved
qemu/tests/virtio_fs_subtest_during_io.py Show resolved Hide resolved
qemu/tests/virtio_fs_subtest_during_io.py Show resolved Hide resolved
@nickzhq nickzhq force-pushed the New_stop_con_vm_during_virtiofs_2223243 branch from 879a5bc to 27fec4b Compare November 17, 2023 03:44
@nickzhq
Copy link
Contributor Author

nickzhq commented Nov 17, 2023

@xiagao @YongxueHong @zhencliu Please take a review, thanks!

@nickzhq nickzhq force-pushed the New_stop_con_vm_during_virtiofs_2223243 branch 4 times, most recently from e0836ad to 479194b Compare November 27, 2023 04:25
@nickzhq
Copy link
Contributor Author

nickzhq commented Nov 27, 2023

@YongxueHong @xiagao Please take a look, thanks!

@nickzhq nickzhq force-pushed the New_stop_con_vm_during_virtiofs_2223243 branch from 479194b to d9ebb7c Compare November 27, 2023 08:29
@nickzhq
Copy link
Contributor Author

nickzhq commented Nov 27, 2023

@YongxueHong @xiagao Please take a look, thanks!

@nickzhq
virtio_fs_utils.py: add @error_context.context_aware to function
b279d52 
Add @error_context.context_aware to function basic_io_test()

Signed-off-by: Houqi (Nick) Zuo <hzuo@redhat.com>
@nickzhq nickzhq force-pushed the New_stop_con_vm_during_virtiofs_2223243 branch 3 times, most recently from 9be090d to 13a3205 Compare November 28, 2023 06:39
xiagao
xiagao approved these changes Nov 28, 2023
View reviewed changes
Copy link
Contributor

@xiagao xiagao left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The test result is pass.
Ack.

YongxueHong
YongxueHong reviewed Nov 28, 2023
View reviewed changes
qemu/tests/virtio_fs_subtest_during_io.py Outdated Show resolved Hide resolved
qemu/tests/virtio_fs_subtest_during_io.py Outdated Show resolved Hide resolved
qemu/tests/virtio_fs_subtest_during_io.py Show resolved Hide resolved
@nickzhq nickzhq force-pushed the New_stop_con_vm_during_virtiofs_2223243 branch from 13a3205 to cbc3f5b Compare November 28, 2023 08:27
YongxueHong
YongxueHong previously approved these changes Nov 28, 2023
View reviewed changes
@nickzhq
stop_con_vm_during_virtiofs: add a new testcase
25abdad 
Signed-off-by: Houqi (Nick) Zuo <hzuo@redhat.com>
@nickzhq nickzhq force-pushed the New_stop_con_vm_during_virtiofs_2223243 branch from cbc3f5b to 25abdad Compare November 28, 2023 08:53
@YongxueHong
Copy link
Contributor

Hi @zhenyzha @fbq815
Please help to review if this PR works on your platform.
Thanks.

@zhencliu
Copy link
Contributor

LGTM

@zhenyzha
Copy link
Contributor

virtio_fs_subtest_during_io PASS (56.71 s)

LGTM.Acked-by: Zhenyu Zhang zhenyzha@redhat.com

fbq815
fbq815 approved these changes Nov 29, 2023
View reviewed changes
Copy link
Contributor

@fbq815 fbq815 left a comment •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

s390x.io-github-autotest-qemu.virtio_fs_subtest_during_io.s390-virtio: PASS (36.73 s)
RESULTS : PASS 1 | ERROR 0 | FAIL 0 | SKIP 0 | WARN 0 | INTERRUPT 0 | CANCEL 0

LGTM

@YongxueHong YongxueHong merged commit 5fb3e30 into autotest:master Nov 29, 2023
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@xiagao xiagao xiagao approved these changes

@YongxueHong YongxueHong YongxueHong approved these changes

@fbq815 fbq815 fbq815 approved these changes

@zhencliu zhencliu zhencliu approved these changes

@zhenyzha zhenyzha zhenyzha approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@nickzhq @xiagao @YongxueHong @zhencliu @zhenyzha @fbq815