Skip to content

Add Ansible integration feature#506

Closed
pablomh wants to merge 1 commit into
theforeman:masterfrom
pablomh:ansible-feature
Closed

Add Ansible integration feature#506
pablomh wants to merge 1 commit into
theforeman:masterfrom
pablomh:ansible-feature

Conversation

@pablomh
Copy link
Copy Markdown
Contributor

@pablomh pablomh commented May 12, 2026

Summary

  • Register ansible feature in features.yaml with foreman_ansible (Foreman),
    ansible (proxy), and foreman_ansible (Hammer CLI) plugins
  • Add smart proxy settings template for the ansible plugin
  • Enable ansible by default in the katello flavor
  • Depends on remote-execution (which transitively enables dynflow)
  • Rename test_foreman_compute_resources to test_foreman_plugin_loaded since
    the parametrized list now includes non-compute plugins

Scope

Ansible is enabled by default in the katello flavor only, matching how other
plugins (katello, remote-execution, rh-cloud) are scoped. Vanilla Foreman installs
can opt in via --add-feature=ansible.

Assumptions

This PR registers the plugin and enables it in the container images. It assumes:

  • The Foreman and Smart Proxy container images already ship foreman_ansible and
    smart_proxy_ansible gems
  • ansible-core is present in the proxy container image
  • SSH key distribution and inventory are handled by REX (dependency)

Test plan

  • foreman_ansible plugin loaded in Foreman (API check)
  • ansible feature exposed on smart proxy /features endpoint
  • "Run Command - Ansible Default" job template seeded
  • ansible listed in foremanctl features
  • Ansible-based REX job execution against a registered host

🤖 Generated with Claude Code

Register foreman_ansible as a feature with Foreman plugin, smart proxy
plugin, and Hammer CLI plugin support. Depends on remote-execution which
transitively enables dynflow on the proxy.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@pablomh pablomh force-pushed the ansible-feature branch from 9bb203f to 6c4f2cb Compare May 12, 2026 14:07
@pablomh
Copy link
Copy Markdown
Contributor Author

pablomh commented May 12, 2026

Dup of #444 :/

@pablomh pablomh closed this May 12, 2026
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.

1 participant