Skip to content

Add katello repos to foremanctl installation doc#4754

Open
arvind4501 wants to merge 1 commit intotheforeman:masterfrom
arvind4501:katello-repo
Open

Add katello repos to foremanctl installation doc#4754
arvind4501 wants to merge 1 commit intotheforeman:masterfrom
arvind4501:katello-repo

Conversation

@arvind4501
Copy link
Copy Markdown
Contributor

@arvind4501 arvind4501 commented Apr 6, 2026

What changes are you introducing?

Add katello-repos package installation step

Why are you introducing these changes? (Explanation, links to references, issues, etc.)

When running foremanctl deploy --add-feature hammer, the installation fails because it tries to install hammer_cli_katello package but cannot find it since the Katello repository is not enabled. Installing
katello-repos-latest.rpm first ensures the Katello repositories are configured before attempting to install Katello-specific packages.

Anything else to add? (Considerations, potential downsides, alternative solutions you have explored, etc.)

Contributor checklists

  • I am okay with my commits getting squashed when you merge this PR.
  • I am familiar with the contributing guidelines.

Please cherry-pick my commits into:

  • Foreman 3.18/Katello 4.20 (Satellite 6.19)
  • Foreman 3.17/Katello 4.19
  • Foreman 3.16/Katello 4.18 (Satellite 6.18; orcharhino 7.6 and 7.7)
  • Foreman 3.15/Katello 4.17
  • Foreman 3.14/Katello 4.16 (Satellite 6.17; orcharhino 7.4; orcharhino 7.5)
  • Foreman 3.13/Katello 4.15 (EL9 only)
  • Foreman 3.12/Katello 4.14 (Satellite 6.16; orcharhino 7.2 on EL9 only; orcharhino 7.3)
  • We do not accept PRs for Foreman older than 3.12.

@github-actions github-actions bot added Needs tech review Requires a review from the technical perspective Needs style review Requires a review from docs style/grammar perspective Needs testing Requires functional testing labels Apr 6, 2026
@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 6, 2026

Copy link
Copy Markdown
Contributor

@archanaserver archanaserver left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Copy Markdown
Contributor

@Gauravtalreja1 Gauravtalreja1 left a comment

Choose a reason for hiding this comment

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

LGTM, just a small change would be nice, could you be specific in your commit message/PR title about which guide needs update?

@arvind4501 arvind4501 changed the title Add katello-repos package installation step Add katello repos to foremanctl installation doc Apr 7, 2026
When running `foremanctl deploy --add-feature hammer`, the installation
fails because it tries to install hammer_cli_katello package but cannot
find it since the Katello repository is not enabled. Installing
katello-repos-latest.rpm first ensures the Katello repositories are
configured before attempting to install Katello-specific packages.

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
@jafiala jafiala self-requested a review April 7, 2026 11:16
Copy link
Copy Markdown
Contributor

@jafiala jafiala left a comment

Choose a reason for hiding this comment

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

LGTM

@jafiala jafiala added tech review done No issues from the technical perspective style review done No issues from docs style/grammar perspective testing done No issues from the functional perspective and removed Needs tech review Requires a review from the technical perspective Needs style review Requires a review from docs style/grammar perspective Needs testing Requires functional testing labels Apr 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

style review done No issues from docs style/grammar perspective tech review done No issues from the technical perspective testing done No issues from the functional perspective

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants