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

rh_kselftests_vm.mm: updates the pattern based on the guest OS version #4256

Merged
merged 1 commit into from
Feb 19, 2025
Merged

rh_kselftests_vm.mm: updates the pattern based on the guest OS version #4256

merged 1 commit into from
Feb 19, 2025

Conversation

mcasquer
Copy link
Contributor

rh_kselftests_vm.mm: updates the pattern based on the guest OS version

The output of the selftests changes, for RHEL 8 versions, it
seems to not contain hastags '#' at the beggining of each line.
Opportunistically, increase the RPM download timeout.

Signed-off-by: mcasquer mcasquer@redhat.com
ID: 3352

@mcasquer mcasquer marked this pull request as ready for review January 31, 2025 08:36
@mcasquer
Copy link
Contributor Author

@zhencliu @yanan-fu could you review this PR? Thanks !

@mcasquer
Copy link
Contributor Author 

 (1/2) Host_RHEL.m9.u6.ovmf.qcow2.virtio_scsi.up.virtio_net.Guest.RHEL.8.10.0.x86_64.io-github-autotest-qemu.rh_kselftests_vm.mm.q35: STARTED
 (1/2) Host_RHEL.m9.u6.ovmf.qcow2.virtio_scsi.up.virtio_net.Guest.RHEL.8.10.0.x86_64.io-github-autotest-qemu.rh_kselftests_vm.mm.q35: PASS (229.54 s)
 (2/2) Host_RHEL.m9.u6.ovmf.qcow2.virtio_scsi.up.virtio_net.Guest.RHEL.9.6.0.x86_64.io-github-autotest-qemu.rh_kselftests_vm.mm.q35: STARTED
 (2/2) Host_RHEL.m9.u6.ovmf.qcow2.virtio_scsi.up.virtio_net.Guest.RHEL.9.6.0.x86_64.io-github-autotest-qemu.rh_kselftests_vm.mm.q35: PASS (306.37 s)
RESULTS    : PASS 2 | ERROR 0 | FAIL 0 | SKIP 0 | WARN 0 | INTERRUPT 0 | CANCEL 0

@mcasquer mcasquer force-pushed the 3352_selftests_rhel8_compatibility branch 2 times, most recently from 3465091 to 85d8dc1 Compare February 12, 2025 10:32
@mcasquer
Copy link
Contributor Author

mcasquer commented Feb 12, 2025 •
edited
Loading

Now the test is compatible with different versions

 (1/2) Host_RHEL.m9.u6.ovmf.qcow2.virtio_scsi.up.virtio_net.Guest.RHEL.8.10.0.x86_64.io-github-autotest-qemu.rh_kselftests_vm.mm.q35: STARTED
 (1/2) Host_RHEL.m9.u6.ovmf.qcow2.virtio_scsi.up.virtio_net.Guest.RHEL.8.10.0.x86_64.io-github-autotest-qemu.rh_kselftests_vm.mm.q35: PASS (148.08 s)
 (2/2) Host_RHEL.m9.u6.ovmf.qcow2.virtio_scsi.up.virtio_net.Guest.RHEL.9.6.0.x86_64.io-github-autotest-qemu.rh_kselftests_vm.mm.q35: STARTED
 (2/2) Host_RHEL.m9.u6.ovmf.qcow2.virtio_scsi.up.virtio_net.Guest.RHEL.9.6.0.x86_64.io-github-autotest-qemu.rh_kselftests_vm.mm.q35: PASS (191.40 s)
RESULTS    : PASS 2 | ERROR 0 | FAIL 0 | SKIP 0 | WARN 0 | INTERRUPT 0 | CANCEL 0

@mcasquer
Copy link
Contributor Author

@zhencliu this is a kindly reminder, please could you review again this PR? Thanks !

@mcasquer
rh_kselftests_vm.mm: updates the pattern based on the guest OS version
3ccc1a3 
The output of the selftests changes, for RHEL 8 versions, it
seems to not contain hastags '#' at the beggining of each line.
Opportunistically, increase the RPM download timeout.

Signed-off-by: mcasquer <mcasquer@redhat.com>
@mcasquer mcasquer force-pushed the 3352_selftests_rhel8_compatibility branch from 85d8dc1 to 3ccc1a3 Compare February 19, 2025 07:04
@mcasquer
Copy link
Contributor Author

Test case passed on both RHEL 8.10 and RHEL 9.6.0

 (1/2) Host_RHEL.m9.u6.ovmf.qcow2.virtio_scsi.up.virtio_net.Guest.RHEL.8.10.0.x86_64.io-github-autotest-qemu.rh_kselftests_vm.mm.q35: STARTED
 (1/2) Host_RHEL.m9.u6.ovmf.qcow2.virtio_scsi.up.virtio_net.Guest.RHEL.8.10.0.x86_64.io-github-autotest-qemu.rh_kselftests_vm.mm.q35: PASS (126.29 s)
 (2/2) Host_RHEL.m9.u6.ovmf.qcow2.virtio_scsi.up.virtio_net.Guest.RHEL.9.6.0.x86_64.io-github-autotest-qemu.rh_kselftests_vm.mm.q35: STARTED
 (2/2) Host_RHEL.m9.u6.ovmf.qcow2.virtio_scsi.up.virtio_net.Guest.RHEL.9.6.0.x86_64.io-github-autotest-qemu.rh_kselftests_vm.mm.q35: PASS (220.99 s)
RESULTS    : PASS 2 | ERROR 0 | FAIL 0 | SKIP 0 | WARN 0 | INTERRUPT 0 | CANCEL 0

yanan-fu
yanan-fu approved these changes Feb 19, 2025
View reviewed changes
Copy link
Contributor

@yanan-fu yanan-fu left a comment

Choose a reason for hiding this comment

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

Ack

@yanan-fu yanan-fu merged commit f4a0e29 into autotest:master Feb 19, 2025
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@zhencliu zhencliu zhencliu approved these changes

@yanan-fu yanan-fu yanan-fu 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.
3 participants
@mcasquer @yanan-fu @zhencliu