Project

General

Profile

Actions

Bug #23905

closed

Default ansible_dir should be ~foreman-proxy

Added by Daniel Lobato Garcia about 6 years ago. Updated almost 6 years ago.

Status:
Closed
Priority:
Normal
Category:
Foreman modules
Target version:
Difficulty:
Triaged:
No
Fixed in Releases:
Found in Releases:

Description

Otherwise, ~foreman-proxy/.ansible.cfg settings are ignored when running Ansible jobs.

Actions #1

Updated by Daniel Lobato Garcia about 6 years ago

  • Project changed from Foreman to Installer
Actions #2

Updated by Daniel Lobato Garcia about 6 years ago

  • Category set to Foreman modules
Actions #3

Updated by Daniel Lobato Garcia about 6 years ago

  • Bugzilla link set to 1590362
Actions #4

Updated by Ewoud Kohl van Wijngaarden about 6 years ago

IMHO it should be configurable. Ideally we can change it to /etc/foreman-proxy/ansible in production for configs and /var/lib/foreman-proxy/ansible for state. Currently we do this via symlinks in RPMs but that's a bit ugly IMHO.

Actions #5

Updated by The Foreman Bot about 6 years ago

  • Status changed from New to Ready For Testing
  • Assignee set to Daniel Lobato Garcia
  • Pull request https://github.com/theforeman/puppet-foreman_proxy/pull/432 added
Actions #6

Updated by Daniel Lobato Garcia about 6 years ago

It's configurable already, both in the installer or manually changing /etc/foreman-proxy/settings.d/ansible.yml. The issue is really just about the default value

Actions #7

Updated by Marek Hulán about 6 years ago

I think the suggested change is valid and already an improvement to current state. Even though config should not live in /usr, that's how we configure foreman-proxy user home and that is where ansible searches for its config. Opening a refactoring issue for later split would be good, but we'd need to handle upgrade path too. Sounds big for something we'll require for 1.18

Actions #8

Updated by Anonymous almost 6 years ago

  • Status changed from Ready For Testing to Closed
  • % Done changed from 0 to 100
Actions #9

Updated by Ewoud Kohl van Wijngaarden almost 6 years ago

  • Target version set to 1.18.0
  • Triaged set to No
  • Fixed in Releases 1.18.0 added
Actions

Also available in: Atom PDF