Bug #17502
closed
Foreman discovery proxy feature installation via puppet not working unless class is specified
Added by N V almost 8 years ago.
Updated almost 8 years ago.
Description
Hello,
The installation of the Foreman Discovery Proxy via Puppet does not work unless the class foreman_proxy::plugin::discovery is included explicitly. This is how my hiera config file for the host looks:
---
classes: [foreman_proxy, 'foreman_proxy::plugin::discovery']
foreman_proxy::dhcp: true
foreman_proxy::tftp: true
foreman_proxy::plugin::discovery::install_images: true
foreman_proxy::plugin::discovery: true
foreman_proxy::dhcp_managed: false
foreman_proxy::tftp_managed: true
foreman_proxy::puppet: false
foreman_proxy::puppetca: false
By "does not work" I mean the puppetrun will work, but won't actually setup the necessary to make discovery work (no fdi image download, no ruby-foreman-discovery package installation, no smart proxy config).
Foreman 1.13.2, Ubuntu 16.04.
- Project changed from Discovery to Installer
- Category changed from Discovery plugin to Foreman modules
Thanks for report, this is for installer component.
- Status changed from New to Need more information
- Translation missing: en.field_release deleted (
207)
I'm not sure I understand the bug report here, the class has to be included to be used. What did you expect?
Dominic Cleal wrote:
I'm not sure I understand the bug report here, the class has to be included to be used. What did you expect?
Definitely true. But I was told in IRC that the install_images class included discovery:
install_images should include that class, see line 26-28 here (https://github.com/theforeman/puppet-foreman_proxy/blob/af05b0791d347451f301a5995e4b150b738966b4/manifests/plugin/discovery.pp#L28
I could well be doing something wrong.
There isn't an install_images class, the class is "foreman_proxy::plugin::discovery". That has to be included by you.
The line you highlighted isn't including a class, it's installing the package inside the foreman_proxy::plugin::discovery class. If you include that class then it should install.
Dominic Cleal wrote:
There isn't an install_images class, the class is "foreman_proxy::plugin::discovery". That has to be included by you.
The line you highlighted isn't including a class, it's installing the package inside the foreman_proxy::plugin::discovery class. If you include that class then it should install.
I guess this can be closed as "non valid" then. Thanks for your clarifications, Dominic.
- Status changed from Need more information to Rejected
Also available in: Atom
PDF
Updated by 
Updated by 
Updated by