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

Fix installation of non-develop katello-devel box #1708

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

Conversation

m-bucher
Copy link
Contributor

The example probably does not work, since the mentioned fix to puppet-katello_devel is probably not available in the yum-repositories.

Requires theforeman/puppet-katello_devel@e276ccb

Comment on lines 21 to 22
- "--katello-devel-extra-plugins theforeman/foreman_rh_cloud"
- "--katello-devel-extra-plugins theforeman/foreman_virt_who_configure"
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Do we really always need this? This breaks the installation of a non-nightly version, because it always takes the latest version.
If we want to always install this (without making it optional), this would also need a possibility to configure the specific SCM-version.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Probably also a question for @chris1984 , because you added this in #1783

I would rather have it in centos9-katello-devel sample definition (see diff), so one can opt-out 😁 .

Copy link
Member

@chris1984 chris1984 Nov 13, 2024

Choose a reason for hiding this comment

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

I am fine with that, we just added it since we were doing a lot of work with virt-who at the time and also now doing work with rh_cloud. Feel free to move it out @m-bucher where it can be opt-out if needed.

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.

3 participants