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

Add candlepin repositories for Katello staging #1743

Merged

Conversation

ekohl
Copy link
Member

@ekohl ekohl commented Dec 7, 2023

No description provided.

@Odilhao
Copy link
Member

Odilhao commented Dec 7, 2023

I think we also need to change the

- name: 'Candlepin {{ katello_repositories_version }} Staging repository'
katello_repositories_version here to candlepin_repositories_version

@ekohl ekohl force-pushed the add-candlepin-repositories-variable branch 3 times, most recently from 22049c1 to d8b3665 Compare December 7, 2023 14:01
roles/candlepin_repositories/defaults/main.yml Outdated Show resolved Hide resolved
Comment on lines +10 to +16
- name: 'Enable pki-core module'
command: dnf module enable -y pki-core
# can't use the `dnf` module for modules without a default stream
# https://github.com/ansible/ansible/issues/56504
# https://github.com/ansible/ansible/issues/64852
args:
creates: /etc/dnf/modules.d/pki-core.module
Copy link
Member

Choose a reason for hiding this comment

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

technically this is not required as people will enabled katello:el8 which pulls that in?

Copy link
Member Author

Choose a reason for hiding this comment

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

For now I will leave it there so you can use candlepin standalone.

ret = forklift_vars
break
else:
possible_versions = set()
for version in reversed(versions['installers']):
if not scenario_os in version['boxes']:
continue
# this is a hack, to be removed once all Katello versions support EL8 in the same manner Foreman does
Copy link
Member

Choose a reason for hiding this comment

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

😆

Candlepin now has its own release cycle and we'l always use the real
repositories. The version is now tracked in versions.yaml, just like
Pulpcore.
@ekohl ekohl force-pushed the add-candlepin-repositories-variable branch from d8b3665 to ae685c5 Compare December 7, 2023 14:12
roles/candlepin_repositories/defaults/main.yml Outdated Show resolved Hide resolved
@Odilhao Odilhao merged commit 1ec3902 into theforeman:master Dec 7, 2023
8 checks passed
@ekohl ekohl deleted the add-candlepin-repositories-variable branch December 7, 2023 15:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants