Nginx support instead of Apache

[dvandendaele]

Hello ,

I would like to use Nginx instead of Apache in the manifest file of your module. Could you make this change ?

[alvagante]

A general suggestion to manage this (as any other resources that is not directly related to the software installation): expose a parameter that let the user override the class where these extra resources (webserver to use, repos to manage or whatever is provided by third modules). In this case it could be something like: $webserver_class = 'puppetexplorer::apache' # The module's default then either provide a puppetexplorer::nginx class to be used as alternative or leave the choice to the module's users. Same thing might be used to manage repos, instead of the booleans manage_yum and manage_apt: $repo_class = 'puppetexplorer::repo'

my2c

[mkrakowitzer]

+1

[roidelapluie]

That issue can be closed:

# [*webserver_class*]
#   Name of the class where the webserver is configured
#   Default: '::puppetexplorer::apache'

https://github.com/spotify/puppet-puppetexplorer/commit/9c6bf2692c05784886f8fccc00184badeb03266c