example42 / puppet-nagios

Puppet module for Nagios
Other
4 stars 18 forks source link

Refactor NRPE, Icinga and Nagios modules. #23

Closed rdrgmnzs closed 11 years ago

alvagante commented 11 years ago

Given that a refactoring of the nagios/icinga, nrpe , nsca , checkmk (still non existing) modules is probably needed, I would try to have each of these modules do its own stuff and be somehow indipendent. Nagios plugins, IMHO, should stay either in a dedicated module or in a nagios::plugins class that can be included anywhere (also where Nagios is not installed). I wouldn't move them inside an NRPE module since you could need Nagios plugins without having NRPE. Also, given this, I would wipe out the redundant nrpe::plugin define. Have to find time to collect the existing PR and ideas, though.