Bug #16669
closedWrong file referenced in puppet_proxy_ssh documentation
Description
https://theforeman.org/manuals/1.12/index.html#4.3.6Puppet
In the "SSH" section the documentation includes the following sed
statements for updating the configuration so that the "Run Puppet" button uses the SSH provider:
sed -i 's!^#\?\(:use_provider:\).*$!\1 puppet_proxy_ssh!' \ /etc/foreman-proxy/settings.d/puppet.yml sed -i 's!^#\?\(:user:\).*$!\1 root!' \ /etc/foreman-proxy/settings.d/puppet.yml sed -i 's!^#\?\(:keyfile:\).*$!\1 /etc/foreman-proxy/id_rsa!' \ /etc/foreman-proxy/settings.d/puppet.yml
The second and third sed
statements are modifying the wrong file. It should be /etc/foreman-proxy/settings.d/puppet_proxy_ssh.yml
:
sed -i 's!^#\?\(:use_provider:\).*$!\1 puppet_proxy_ssh!' \ /etc/foreman-proxy/settings.d/puppet.yml sed -i 's!^#\?\(:user:\).*$!\1 root!' \ /etc/foreman-proxy/settings.d/puppet_proxy_ssh.yml sed -i 's!^#\?\(:keyfile:\).*$!\1 /etc/foreman-proxy/id_rsa!' \ /etc/foreman-proxy/settings.d/puppet_proxy_ssh.yml
Updated by Jason Nance over 8 years ago
It seems that perhaps this configuration file was split at some point and the docs not updated for this change in paradigm as later on in the same section it references the :user:
and :keyfile:
being in the puppet.yml
file. Look for "The configuration file should now contain" in the same section. So I believe there is a bit of reworking needed for this section.
Updated by Dominic Cleal over 8 years ago
- Status changed from New to Ready For Testing
- Assignee set to Dominic Cleal
- Pull request https://github.com/theforeman/theforeman.org/pull/709 added
Updated by Dominic Cleal over 8 years ago
- Status changed from Ready For Testing to Closed
- % Done changed from 0 to 100
Applied in changeset theforemanorg|4760e5c40380d9f947f8ba05323923ab286946a0.