-
Notifications
You must be signed in to change notification settings - Fork 148
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
Foreman Client el10 build support #11532
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Conceptually this looks fine. Just remarks about the list of packages.
<packagelist> | ||
<packagereq type="default">foreman-client-release</packagereq> | ||
<packagereq type="default">foreman_ygg_migration</packagereq> | ||
<packagereq type="default">foreman_ygg_worker</packagereq> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@adamruzicka this fails to build on EL10:
https://download.copr.fedorainfracloud.org/results/evgeni/el10/centos-stream-10-x86_64/08343540-foreman_ygg_worker/builder-live.log.gz
Shouldn't block this PR, but something that will end up on your plate one day ;)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Copying discussion from Matrix: after #11493 is merged we can investigate using the proper go macros to build. Those weren't available on EL7.
9b0f47c
to
15d7b31
Compare
Updated :D |
I have built katello-host-tools katello-pull-transport-migrate rubygem-foreman_scap_client foreman-release |
No description provided.