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

block_hotplug_check:Update pci search pattern #3797

Merged
merged 1 commit into from
Aug 30, 2023

Conversation

SiboWang1997
Copy link
Contributor

Updates the case logic using numerical ID, as test point due to name will be changed without notice. In order to support 0.9 and 1.0 virtio devices, the pattern now contains both IDs.

ID: 2230552
Signed-off-by: Sibo Wang [email protected]

@SiboWang1997
Copy link
Contributor Author

python3 ConfigTest.py --guestname=RHEL.9.3.0..page_64k --platform=aarch64 --machines=arm64-pci --netdst=virbr0 --driveformat=virtio_blk --nicmodel=virtio_net --mem=8192 --vcpu=4 --testcase=block_hotplug_check --clone=no --debug

(1/1) Host_RHEL.m9.u3.qcow2.virtio_blk.up.virtio_net.Guest.RHEL.9.3.0.aarch64.page_64k.io-github-autotest-qemu.block_hotplug_check.arm64-pci: STARTED
(1/1) Host_RHEL.m9.u3.qcow2.virtio_blk.up.virtio_net.Guest.RHEL.9.3.0.aarch64.page_64k.io-github-autotest-qemu.block_hotplug_check.arm64-pci: PASS (807.29 s)
RESULTS : PASS 1 | ERROR 0 | FAIL 0 | SKIP 0 | WARN 0 | INTERRUPT 0 | CANCEL 0
JOB HTML : /root/avocado/job-results/job-2023-08-11T03.44-e5a74a8/results.html
JOB TIME : 819.51 s

@SiboWang1997
Copy link
Contributor Author

Hi @XueqiangWei @qingwangrh @zhenyzha , could you please help to review this patch? Thanks a lot. (This patch refers to #3768 's solution )

@zhenyzha
Copy link
Contributor

LGTM.Acked-by: Zhenyu Zhang [email protected]

git am 3797.patch
Applying: block_hotplug_check:Update pci search pattern

(1/1) Host_RHEL.m9.u3.qcow2.virtio_blk.up.virtio_net.Guest.RHEL.9.3.0.aarch64.page_64k.io-github-autotest-qemu.block_hotplug_check.arm64-pci: PASS (861.35 s)

Updates the case logic using numerical ID, as test point due to name
will be changed without notice. In order to support 0.9 and 1.0 virtio
devices, the pattern now contains both IDs.

Signed-off-by: Sibo Wang <[email protected]>
@SiboWang1997
Copy link
Contributor Author

Hi @XueqiangWei @qingwangrh could you please help to review this patch? Thanks a lot.

@XueqiangWei
Copy link
Contributor

LGTM,
Acked-by: [email protected]

@qingwangrh
Copy link
Contributor

python3 ConfigTest.py --guestname=RHEL.9.3.0 --netdst=virbr0 --driveformat=virtio_blk --nicmodel=virtio_net --mem=8192 --vcpu=4 --testcase=block_hotplug_check --clone=no --debug --firmware=default_bios

(1/1) Host_RHEL.m9.u3.qcow2.virtio_blk.up.virtio_net.Guest.RHEL.9.3.0.x86_64.io-github-autotest-qemu.block_hotplug_check.q35: STARTED
(1/1) Host_RHEL.m9.u3.qcow2.virtio_blk.up.virtio_net.Guest.RHEL.9.3.0.x86_64.io-github-autotest-qemu.block_hotplug_check.q35: PASS (983.57 s)
RESULTS : PASS 1 | ERROR 0 | FAIL 0 | SKIP 0 | WARN 0 | INTERRUPT 0 | CANCEL 0

LGTM

@XueqiangWei XueqiangWei merged commit ae24501 into autotest:master Aug 30, 2023
6 checks passed
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.

5 participants