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

Clean up problematic rpm macro, install skopeo package for centos9 #2292

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

dmick
Copy link
Member

@dmick dmick commented Nov 13, 2024

couple small changes to the ansible for Jenkins builders:

  1. don't set dist in rpmmacros; it was causing el9 packages to be labeled el8, and isn't necessary at all
  2. add skopeo to installed packages (for container builds).

The package code ought to be cleaned up so that 'epel_packages' is what it says it is.

This caused some really weird breakage where el8 was being set
on el9 machines, and it's no longer necessary (I don't know why it
was ever there)

Signed-off-by: Dan Mick <[email protected]>
arguably we should fix 'epel_packages' because I think none of those
are in epel anymore, but in this file it really means 'packages
that are release-specific'

Signed-off-by: Dan Mick <[email protected]>
@dmick dmick requested a review from zmc November 13, 2024 03:10
@dmick
Copy link
Member Author

dmick commented Nov 26, 2024

@zmc @djgalloway ping

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