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

e2e: switch default test distro to fedora/40-cloud-base. #375

Merged

Conversation

klihub
Copy link
Collaborator

@klihub klihub commented Oct 4, 2024

Fedora 37 is EOL. Switch default test distro to fedora 40 and use cloud base image.

@klihub
Copy link
Collaborator Author

klihub commented Oct 4, 2024

@askervin @fmuyassarov @kad @pfl PTAL.

We need python-{selinux,packaging} with distro=fedora/40-cloud-image.

Signed-off-by: Krisztian Litkey <[email protected]>
@klihub klihub force-pushed the devel/e2e/default-to-fedora-40-cloud-base branch from ee5279c to ab25ea1 Compare October 4, 2024 15:11
Copy link
Collaborator

@kad kad left a comment

Choose a reason for hiding this comment

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

LGTM

Comment on lines +146 to +155
pkg:
- iproute-tc
- grubby
state: present
update_cache: true
- name: Extra python3 rpm packages for ansible
ansible.builtin.dnf:
pkg:
- python3-libselinux
- python3-packaging
Copy link
Collaborator

@fmuyassarov fmuyassarov Oct 5, 2024

Choose a reason for hiding this comment

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

I would merge these two tasks into one and create a single list of requires packages.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

The 2nd one is conditional not necessary (and neither possible) on our older Fedora distros.

@fmuyassarov fmuyassarov merged commit ed00b79 into containers:main Oct 5, 2024
3 checks passed
@klihub klihub deleted the devel/e2e/default-to-fedora-40-cloud-base branch October 5, 2024 14:14
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.

4 participants