Skip to content

Switch setup-repositories only to Foreman, plugins and Katello - #784

Merged
ehelms merged 1 commit into
theforeman:masterfrom
ehelms:only-install-foreman-katello-plugins
Aug 26, 2026
Merged

Switch setup-repositories only to Foreman, plugins and Katello#784
ehelms merged 1 commit into
theforeman:masterfrom
ehelms:only-install-foreman-katello-plugins

Conversation

@ehelms

@ehelms ehelms commented Aug 25, 2026

Copy link
Copy Markdown
Member

Why are you introducing these changes? (Problem description, related links)

To help ensure the system reflects as close to production as possible. The Pulp repositories can cause clashes with system dependencies.

What are the changes introduced in this pull request?

  • Only deploy Foreman, plugins and katello repositories

How to test this pull request

Steps to reproduce:

  • Look at CI

Checklist

  • Tests added/updated (if applicable)
  • Documentation updated (if applicable)

@ehelms
ehelms force-pushed the only-install-foreman-katello-plugins branch 2 times, most recently from b903f0f to 18fd64f Compare August 26, 2026 01:17
name: katello
description: Katello repository
gpgcheck: false
baseurl: https://yum.theforeman.org/katello/{{ setup_repositories_katello_version }}/katello/el{{ ansible_facts['distribution_major_version'] }}/x86_64/

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Can we be ready for the future, even if I can only dream of it now?

Suggested change
baseurl: https://yum.theforeman.org/katello/{{ setup_repositories_katello_version }}/katello/el{{ ansible_facts['distribution_major_version'] }}/x86_64/
baseurl: https://yum.theforeman.org/katello/{{ setup_repositories_katello_version }}/katello/el{{ ansible_facts['distribution_major_version'] }}/$basearch/

foreman_openvox_repositories_version: "8"
- name: Setup repository for Foreman {{ setup_repositories_version }}
ansible.builtin.package:
name: https://yum.theforeman.org/releases/{{ setup_repositories_version }}/el{{ ansible_facts['distribution_major_version'] }}/x86_64/foreman-release.rpm

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Can we be ready for the future, even if I can only dream of it now?

Suggested change
name: https://yum.theforeman.org/releases/{{ setup_repositories_version }}/el{{ ansible_facts['distribution_major_version'] }}/x86_64/foreman-release.rpm
name: https://yum.theforeman.org/releases/{{ setup_repositories_version }}/el{{ ansible_facts['distribution_major_version'] }}/$basearch/foreman-release.rpm

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Nope! Because this is Ansible installing an RPM, not configuring a yum repo and it doesn't interpret that :)

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

That makes me sad, but you're right.

@@ -0,0 +1,4 @@
---
setup_repositories_version: nightly
setup_repositories_katello_version: nightly

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Katello now matches versions with Foreman. I think we can ignore any pre-5.0 version so can we drop this variable?

The Pulp RPM repositories are clashing with system dependencies.
@ehelms
ehelms force-pushed the only-install-foreman-katello-plugins branch from 18fd64f to ab9fdca Compare August 26, 2026 13:25
@ehelms

ehelms commented Aug 26, 2026

Copy link
Copy Markdown
Member Author

As the failures are now seen related to #734 I am going to merge this as it fixes the deployment issue and will allow having just a single set of failures to figure out.

@ehelms
ehelms merged commit f349b9f into theforeman:master Aug 26, 2026
7 of 18 checks passed
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