Skip to content

Update the case's epel repos for RHEL10 testing #4308

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

peixiu
Copy link
Contributor

@peixiu peixiu commented May 14, 2025

Since from RHEL10. the cpuid package is excludes from epel_10.repo. So change the repo link to epel_9 for RHEL version >=10

ID: 3306

@peixiu
Copy link
Contributor Author

peixiu commented May 14, 2025

python3 ConfigTest.py --testcase=hv_flag_cpuid_check,hv_avic,hv_enforce_cpuid_msr_check --guestname=RHEL.10.0 --machines=q35 --firmware=ovmf --clone=yes

(1/5) Host_RHEL.m10.u1.ovmf.qcow2.virtio_scsi.up.virtio_net.Guest.RHEL.10.0.x86_64.io-github-autotest-qemu.unattended_install.cdrom.extra_cdrom_ks.default_install.aio_threads.q35: STARTED
(1/5) Host_RHEL.m10.u1.ovmf.qcow2.virtio_scsi.up.virtio_net.Guest.RHEL.10.0.x86_64.io-github-autotest-qemu.unattended_install.cdrom.extra_cdrom_ks.default_install.aio_threads.q35: PASS (593.75 s)
(2/5) Host_RHEL.m10.u1.ovmf.qcow2.virtio_scsi.up.virtio_net.Guest.RHEL.10.0.x86_64.io-github-autotest-qemu.hv_flag_cpuid_check.hv_tlbflush_ext.q35: STARTED
(2/5) Host_RHEL.m10.u1.ovmf.qcow2.virtio_scsi.up.virtio_net.Guest.RHEL.10.0.x86_64.io-github-autotest-qemu.hv_flag_cpuid_check.hv_tlbflush_ext.q35: PASS (200.75 s)
(3/5) Host_RHEL.m10.u1.ovmf.qcow2.virtio_scsi.up.virtio_net.Guest.RHEL.10.0.x86_64.io-github-autotest-qemu.hv_flag_cpuid_check.hv_tlbflush_direct.q35: STARTED
(3/5) Host_RHEL.m10.u1.ovmf.qcow2.virtio_scsi.up.virtio_net.Guest.RHEL.10.0.x86_64.io-github-autotest-qemu.hv_flag_cpuid_check.hv_tlbflush_direct.q35: PASS (118.20 s)
(4/5) Host_RHEL.m10.u1.ovmf.qcow2.virtio_scsi.up.virtio_net.Guest.RHEL.10.0.x86_64.io-github-autotest-qemu.hv_avic.q35: STARTED
(4/5) Host_RHEL.m10.u1.ovmf.qcow2.virtio_scsi.up.virtio_net.Guest.RHEL.10.0.x86_64.io-github-autotest-qemu.hv_avic.q35: PASS (40.90 s)
(5/5) Host_RHEL.m10.u1.ovmf.qcow2.virtio_scsi.up.virtio_net.Guest.RHEL.10.0.x86_64.io-github-autotest-qemu.hv_enforce_cpuid_msr_check.q35: STARTED
(5/5) Host_RHEL.m10.u1.ovmf.qcow2.virtio_scsi.up.virtio_net.Guest.RHEL.10.0.x86_64.io-github-autotest-qemu.hv_enforce_cpuid_msr_check.q35: PASS (91.45 s)

@yanan-fu Could you help to review this patch? I updated the case's epel repo, Using epel-9.repo for >=RHEL9, since the cpuid package is excludes in epe-10.repo from RHEL10.0 start.

Thanks~
Peixiu

Since from RHEL10. the cpuid package is excludes from epel_10.repo.
So change the repo link to epel_9 for RHEL version >=10

Signed-off-by: Peixiu Hou <[email protected]>
@@ -7,5 +7,6 @@
cpuid_chk_cmd = "rpm -qa | grep cpuid"
cpuid_pkg = "cpuid"
RHEL:
repo_install_cmd = "dnf install -y https://dl.fedoraproject.org/pub/epel/epel-release-latest-%s.noarch.rpm"
#set workaround from RHEL10.0 since no cpuid package includes in epel-10.repo
repo_install_cmd = "dnf install -y https://dl.fedoraproject.org/pub/epel/epel-release-latest-9.noarch.rpm"
Copy link
Contributor

Choose a reason for hiding this comment

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

@peixiu QEMU 6.2 is available for RHEL8, do you plan to remove the rhel8 support ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@yanan-fu yes, our testing focus on the recent rhel9 and rhel10 version.

Copy link
Contributor

Choose a reason for hiding this comment

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

@peixiu This test case configuration does not have the limitation of exclude RHEL-8.

@yanan-fu
Copy link
Contributor

@peixiu Use epel9 for RHEL8+ is risky in the future for pkg conflict issue.
I do not know the reason of removing cpuid outof the epel-10, but i would suggest raise this in the epel side to see if it can resolved totally instead of using a workaround, refer:
https://docs.fedoraproject.org/en-US/epel/epel-package-request/

@peixiu
Copy link
Contributor Author

peixiu commented May 26, 2025

@peixiu Use epel9 for RHEL8+ is risky in the future for pkg conflict issue. I do not know the reason of removing cpuid outof the epel-10, but i would suggest raise this in the epel side to see if it can resolved totally instead of using a workaround, refer: https://docs.fedoraproject.org/en-US/epel/epel-package-request/

Thanks @yanan-fu ~
I filed a bug as the document recommand: https://bugzilla.redhat.com/show_bug.cgi?id=2368495
I'm not sure if there're some one response our request, we can wait some time for this bug status~

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.

2 participants