Skip to content

Conversation

maximiliankolb
Copy link
Contributor

@maximiliankolb maximiliankolb commented Jul 24, 2025

What changes are you introducing?

Remove docs on how to configure configuring hosts with foreman_ansible by overriding Ansible variables of imported Ansible roles with host parameters because it does not work.

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

I tried to reproduce this on Foreman 3.14 but could not get it to run.

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

draft PR until reproduced by someone else

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.15/Katello 4.17
  • Foreman 3.14/Katello 4.16 (Satellite 6.17)
  • 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)
  • Foreman 3.11/Katello 4.13 (orcharhino 6.11 on EL8 only; orcharhino 7.0 on EL8+EL9; orcharhino 7.1 with Leapp)
  • Foreman 3.10/Katello 4.12
  • Foreman 3.9/Katello 4.11 (Satellite 6.15; orcharhino 6.8/6.9/6.10)
  • We do not accept PRs for Foreman older than 3.9.

Review checklists

Tech review (performed by an Engineer who did not author the PR; can be skipped if tech review is unnecessary):

  • The PR documents a recommended, user-friendly path.
  • The PR removes steps that have been made unnecessary or obsolete.
  • Any steps introduced or updated in the PR have been tested to confirm that they lead to the documented end result.

Style review (by a Technical Writer who did not author the PR):

  • The PR conforms with the team's style guidelines.
  • The PR introduces documentation that describes a user story rather than a product feature.

@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 Jul 24, 2025
. In the {ProjectWebUI}, navigate to *Hosts* > *All Hosts*.
. Select your host.
. On the *Ansible* tab, select the *Variables* tab.
. Click the pencil icon to edit the value of the variable.

Choose a reason for hiding this comment

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

suggestion: This way of overriding a variable is an alternative way to create a matcher (described in this file in the lines before the diff). While we are at it, maybe we could make this clear? Otherwise people might be confused as to how these two features interact (e.g., if I create a matcher AND set the variable via the host's ansible variables tab, which one has precedence)

@maximiliankolb maximiliankolb force-pushed the remove_ansible_roles_with_host_parameters branch from 3f554b2 to d0b6e93 Compare September 18, 2025 11:43
Copy link
Contributor Author

@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.

Rebased to HEAD of "master".

@maximiliankolb maximiliankolb changed the title Remove ansible roles with host parameters How to override Ansible variables Sep 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Needs style review Requires a review from docs style/grammar perspective Needs tech review Requires a review from the technical perspective Needs testing Requires functional testing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants