Foreman and Ansible documentation improvements needed

flechoide · March 24, 2020, 2:15pm

Hello, Im following the documentation in order to setup a simple foreman enc for ansible, it seems to be things that just dont fit.

Taking fopreman 1.24 with plugin ansible 3.x : https://www.theforeman.org/plugins/foreman_ansible/3.x/index.html#2.Installation

You are told to set some configs at ansible.cfg

[defaults]
callback_whitelist = foreman

Also the call_back foreman is incorrect it should something like thjat , and should be noted that the right info can be found that way : egrep ‘ssl|port’ /etc/foreman-proxy/settings.yml | grep -v ‘^#’
[callback_foreman]
url = ‘https://foremanansible.mydomain.lab’
ssl_cert = /etc/puppetlabs/puppet/ssl/certs/$(hostname -f)
ssl_key = /etc/puppetlabs/puppet/ssl/private_keys/$(hostname -f)
verify_certs = /etc/puppetlabs/puppet/ssl/certs/ca.pem

At point 4 the problems arise, it seems that the gif shows some configs that are not well documented (Ivebeen unablke to find docs for them) .

Also the line bin_ansible_callbacks=True should be noted before point 4

The ansible plugin for foreman (foreman_callback.py) seems to be represented with several diferent versions . For what I think the right version is that one :

github.com

theforeman/foreman_ansible/blob/master/extras/foreman_callback.py

# (C) 2015, 2016 Daniel Lobato <elobatocs@gmail.com>

# This file is part of Ansible
#
# Ansible is free software: you can redistribute it and/or modify
# it under the terms of the GNU General Public License as published by
# the Free Software Foundation, either version 3 of the License, or
# (at your option) any later version.
#
# Ansible is distributed in the hope that it will be useful,
# but WITHOUT ANY WARRANTY; without even the implied warranty of
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
# GNU General Public License for more details.
#
# You should have received a copy of the GNU General Public License
# along with Ansible.  If not, see <http://www.gnu.org/licenses/>.

import os
from datetime import datetime
from collections import defaultdict

This file has been truncated. show original

It should be noted in the docs that you need to retrieve the foreman.py from github

lzap · March 27, 2020, 12:23pm

Hello, thanks for the report. Mind creating a PR in our documentation repo?

flechoide · April 8, 2020, 11:11am

Hello Izap, Im sorry for the delay, I didnt know of such an approach was available, I will take a look of course