Skip to content

Add auxiliary IDs for SAT-41410#4708

Open
adamruzicka wants to merge 1 commit intotheforeman:3.18from
adamruzicka:aux
Open

Add auxiliary IDs for SAT-41410#4708
adamruzicka wants to merge 1 commit intotheforeman:3.18from
adamruzicka:aux

Conversation

@adamruzicka
Copy link
Copy Markdown
Contributor

What changes are you introducing?

Adding auxiliary IDs for compatibility. Namely addressing all the changes done in RedHatSatellite/foreman_theme_satellite#302 with the intention of reverting that code change once this goes in.

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

The documentation structure changed after the release.

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

There were some section renames, which will require setting up redirects in the Satellite documentation, but that has to be handled outside of this PR.

For the sake of completeness

Redirect from Redirect to
/managing_hosts/configuring_and_setting_up_remote_jobs_managing-hosts /managing_hosts/configuring-and-setting-up-remote-jobs
/managing_hosts/monitoring-hosts-by-using-insights /managing_hosts/monitoring-hosts-by-using-red-hat-lightspeed-in-cloud

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.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 Mar 11, 2026
@github-actions
Copy link
Copy Markdown

github-actions bot commented Mar 11, 2026

Copy link
Copy Markdown
Contributor

@maximiliankolb maximiliankolb left a comment

Choose a reason for hiding this comment

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

diff LGTM

@maximiliankolb maximiliankolb added style review done No issues from docs style/grammar perspective and removed Needs style review Requires a review from docs style/grammar perspective labels Mar 12, 2026
@adamruzicka adamruzicka force-pushed the aux branch 2 times, most recently from 7a60a09 to 03fa2e9 Compare March 17, 2026 14:07
@maximiliankolb
Copy link
Copy Markdown
Contributor

image

I think something went wrong here. Please try to rebase your changes onto HEAD of "3.18".

@adamruzicka adamruzicka force-pushed the aux branch 2 times, most recently from 1076a21 to 7d81eaf Compare March 18, 2026 07:48
@adamruzicka
Copy link
Copy Markdown
Contributor Author

In retrospect, naming three different branches across two repositories aux and aux-id wasn't the greatest of ideas.

Copy link
Copy Markdown
Contributor

@maximiliankolb maximiliankolb left a comment

Choose a reason for hiding this comment

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

Thanks Adam, diff LGTM!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants