Foreman » Installer

I assume all we really need here is an array of extra packages to install?

Greg Sutcliffe wrote:

I assume all we really need here is an array of extra packages to install?

I think we should provide more than that, bearing in mind the kafo installer interface, i.e. --foreman-libvirt=true is clearer than --foreman-extra-packages=foreman-libvirt.

For plugins, I don't think a package install is sufficient. I'd say an additional Puppet class that can be enabled is needed (especially if it can use kafo's module + parameter support), as we'll probably want to provide configuration as well as just a package. Consider column_view as an example, which has an associated config file. Even for plugins that don't have config yet, we can seamlessly add it later.

plugins:
- arekatello
- bootdisk
- discovery
- foreman-hooks
- foreman-setup

Some other plugins worth including in the installer:

puppetdb_foreman
foreman_chef

There's already foreman::plugin::puppetdb: https://github.com/theforeman/puppet-foreman/blob/master/manifests/plugin/puppetdb.pp

Daniel - I'm not sure puppetdb or foreman_chef need to be installed by default. As Ewoud says, we already have the puppetdb one in the repo, I can easily add the foreman_chef entry as well, so you can call it if you like :)

No, not by default, but they should be available in the installer.