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.

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

Add libblkio release test #3780

Merged

Conversation

qingwangrh
Copy link
Contributor

It will verify the source code of
libblkio is working and test the main
driver virtio-blk-vhost-vdpa

ID:2222155

@qingwangrh
Copy link
Contributor Author

python ConfigTest.py --testcase=block_libblkio_release --guestname=RHEL.9.3.0 --clone=no --firmware=default_bios --nrepeat=5

(1/5) repeat1.Host_RHEL.m9.u3.qcow2.virtio_scsi.up.virtio_net.Guest.RHEL.9.3.0.x86_64.io-github-autotest-qemu.block_libblkio_release.q35: STARTED
(1/5) repeat1.Host_RHEL.m9.u3.qcow2.virtio_scsi.up.virtio_net.Guest.RHEL.9.3.0.x86_64.io-github-autotest-qemu.block_libblkio_release.q35: PASS (69.05 s)
(2/5) repeat2.Host_RHEL.m9.u3.qcow2.virtio_scsi.up.virtio_net.Guest.RHEL.9.3.0.x86_64.io-github-autotest-qemu.block_libblkio_release.q35: STARTED
(2/5) repeat2.Host_RHEL.m9.u3.qcow2.virtio_scsi.up.virtio_net.Guest.RHEL.9.3.0.x86_64.io-github-autotest-qemu.block_libblkio_release.q35: PASS (68.50 s)
(3/5) repeat3.Host_RHEL.m9.u3.qcow2.virtio_scsi.up.virtio_net.Guest.RHEL.9.3.0.x86_64.io-github-autotest-qemu.block_libblkio_release.q35: STARTED
(3/5) repeat3.Host_RHEL.m9.u3.qcow2.virtio_scsi.up.virtio_net.Guest.RHEL.9.3.0.x86_64.io-github-autotest-qemu.block_libblkio_release.q35: PASS (68.17 s)
(4/5) repeat4.Host_RHEL.m9.u3.qcow2.virtio_scsi.up.virtio_net.Guest.RHEL.9.3.0.x86_64.io-github-autotest-qemu.block_libblkio_release.q35: STARTED
(4/5) repeat4.Host_RHEL.m9.u3.qcow2.virtio_scsi.up.virtio_net.Guest.RHEL.9.3.0.x86_64.io-github-autotest-qemu.block_libblkio_release.q35: PASS (69.07 s)
(5/5) repeat5.Host_RHEL.m9.u3.qcow2.virtio_scsi.up.virtio_net.Guest.RHEL.9.3.0.x86_64.io-github-autotest-qemu.block_libblkio_release.q35: STARTED
(5/5) repeat5.Host_RHEL.m9.u3.qcow2.virtio_scsi.up.virtio_net.Guest.RHEL.9.3.0.x86_64.io-github-autotest-qemu.block_libblkio_release.q35: PASS (68.55 s)
RESULTS : PASS 5 | ERROR 0 | FAIL 0 | SKIP 0 | WARN 0 | INTERRUPT 0 | CANCEL 0

@qingwangrh
Copy link
Contributor Author

@XueqiangWei @zhenyzha @fbq815 @maxujun Please help to review, thanks.

@zhenyzha
Copy link
Contributor

git am 3780.patch
Applying: Add libblkio release test
.git/rebase-apply/patch:44: new blank line at EOF.
+
warning: 1 line adds whitespace errors.

@qingwangrh qingwangrh force-pushed the 2222155_add_libbklid_release branch from 0a5f145 to 16b1d27 Compare July 19, 2023 12:16
@qingwangrh
Copy link
Contributor Author

git am 3780.patch Applying: Add libblkio release test .git/rebase-apply/patch:44: new blank line at EOF. + warning: 1 line adds whitespace errors.

updated

@XueqiangWei
Copy link
Contributor

(1/5) repeat1.Host_RHEL.m9.u3.ovmf.qcow2.virtio_scsi.up.virtio_net.Guest.RHEL.9.3.0.x86_64.io-github-autotest-qemu.block_libblkio_release.q35: STARTED
(1/5) repeat1.Host_RHEL.m9.u3.ovmf.qcow2.virtio_scsi.up.virtio_net.Guest.RHEL.9.3.0.x86_64.io-github-autotest-qemu.block_libblkio_release.q35: PASS (210.49 s)
(2/5) repeat2.Host_RHEL.m9.u3.ovmf.qcow2.virtio_scsi.up.virtio_net.Guest.RHEL.9.3.0.x86_64.io-github-autotest-qemu.block_libblkio_release.q35: STARTED
(2/5) repeat2.Host_RHEL.m9.u3.ovmf.qcow2.virtio_scsi.up.virtio_net.Guest.RHEL.9.3.0.x86_64.io-github-autotest-qemu.block_libblkio_release.q35: PASS (79.14 s)
(3/5) repeat3.Host_RHEL.m9.u3.ovmf.qcow2.virtio_scsi.up.virtio_net.Guest.RHEL.9.3.0.x86_64.io-github-autotest-qemu.block_libblkio_release.q35: STARTED
(3/5) repeat3.Host_RHEL.m9.u3.ovmf.qcow2.virtio_scsi.up.virtio_net.Guest.RHEL.9.3.0.x86_64.io-github-autotest-qemu.block_libblkio_release.q35: PASS (77.80 s)
(4/5) repeat4.Host_RHEL.m9.u3.ovmf.qcow2.virtio_scsi.up.virtio_net.Guest.RHEL.9.3.0.x86_64.io-github-autotest-qemu.block_libblkio_release.q35: STARTED
(4/5) repeat4.Host_RHEL.m9.u3.ovmf.qcow2.virtio_scsi.up.virtio_net.Guest.RHEL.9.3.0.x86_64.io-github-autotest-qemu.block_libblkio_release.q35: PASS (78.80 s)
(5/5) repeat5.Host_RHEL.m9.u3.ovmf.qcow2.virtio_scsi.up.virtio_net.Guest.RHEL.9.3.0.x86_64.io-github-autotest-qemu.block_libblkio_release.q35: STARTED
(5/5) repeat5.Host_RHEL.m9.u3.ovmf.qcow2.virtio_scsi.up.virtio_net.Guest.RHEL.9.3.0.x86_64.io-github-autotest-qemu.block_libblkio_release.q35: PASS (80.01 s)

@maxujun
Copy link
Contributor

maxujun commented Jul 25, 2023

(1/1) Host_RHEL.m8.u9.product_rhel.qcow2.virtio_blk.up.virtio_net.Guest.RHEL.9.3.0.ppc64le.io-github-autotest-qemu.block_libblkio_release: STARTED
(1/1) Host_RHEL.m8.u9.product_rhel.qcow2.virtio_blk.up.virtio_net.Guest.RHEL.9.3.0.ppc64le.io-github-autotest-qemu.block_libblkio_release: CANCEL: Got host qemu version:6.2.0-36, which is not in [8.0.0,) (2.93 s)

LGTM, Acked.

@fbq815
Copy link
Contributor

fbq815 commented Jul 25, 2023

s390x don't have this module in kvm, please skip s390x for this mr

@zhenyzha
Copy link
Contributor

aarch64 doesn't have this module in kvm, please skip aarch64 for this mr too.

It will verify the source code of
libblkio is working and test the main
driver virtio-blk-vhost-vdpa

Signed-off-by: qingwangrh <[email protected]>
@qingwangrh qingwangrh force-pushed the 2222155_add_libbklid_release branch from 16b1d27 to cd494b9 Compare July 26, 2023 02:33
@XueqiangWei
Copy link
Contributor

(1/1) Host_RHEL.m9.u3.ovmf.qcow2.virtio_scsi.up.virtio_net.Guest.RHEL.9.3.0.x86_64.io-github-autotest-qemu.block_libblkio_release.q35: PASS (85.45 s)

@XueqiangWei XueqiangWei merged commit fdef73f into autotest:master Jul 26, 2023
6 checks passed
@SiboWang1997
Copy link
Contributor

SiboWang1997 commented Aug 7, 2023

Hi @XueqiangWei, aarch64 and s390x don't have this module in kvm, but it seems that they are not filtered out, could you review it again?

Cc: @zhencliu @YongxueHong

@XueqiangWei
Copy link
Contributor

Hi @XueqiangWei, aarch64 and s390x don't have this module in kvm, but it seems that they are not filtered out, could you review it again?

Thank you for pointing out it, since this pullreq has been merged. Let's try it in a new pullreq. cc @qingwangrh

@wq3721
Copy link

wq3721 commented Aug 9, 2023

Hi @XueqiangWei, aarch64 and s390x don't have this module in kvm, but it seems that they are not filtered out, could you review it again?

Cc: @zhencliu @YongxueHong

suggest filter out in kar loop.

@XueqiangWei
Copy link
Contributor

Hi @XueqiangWei, aarch64 and s390x don't have this module in kvm, but it seems that they are not filtered out, could you review it again?
Cc: @zhencliu @YongxueHong

suggest filter out in kar loop.

Yes, @qingwangrh is more inclined to use this way. He has discussed it with @zhenyzha and will discuss it with @SiboWang1997. Thank you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants