Fix incorrect template data-type and add tests.

Pull Request (PR) description

Remove reference to recurse => true in README, as that option was removed in Commit 0bf84794ac98b55b5e54e72b7b3f56a129ae5b80

This Pull Request (PR) fixes the following issues

N/A

Pull Request (PR) description

Allow allow_bash_command_substitution to be set.

This Pull Request (PR) fixes the following issues

config manifest sets the value to a string of 0 or 1, but the template expects a Boolean. Since we need to allow for undef value, best to keep it Boolean all the way through.

Error: Evaluation Error: Error while evaluating a Function Call, lambda: parameter 'allow_bash_command_substitution' expects a value of type Undef or Boolean, got String (file: /etc/puppetlabs/code/environments/nrpe_update/modules/nrpe/manifests/config.pp, line: 24, column: 16) on node localhost.localdomain


Pull Request (PR) description

Allow to use sudo outside of /usr/bin. It usefull for FreeBSD compatibility. There is also a patch to update OS compatibility list.

This Pull Request (PR) fixes the following issues

N/A

modulesync 5.1.0

<!--
Thank you for contributing to this project!

• This project has a Contributor Code of Conduct: https://voxpupuli.org/coc/
• Please check that here is no existing issue or PR that addresses your problem.
• Our vulnerabilities reporting process is at https://voxpupuli.org/security/

-->

Pull Request (PR) description

Packagestyle not supported under FreeBSD yet.
Added the right package style with FreeBSD NRPEv3 support.
Added new variable to setup listenqueuesize for huge set of monitoring commands
Include description of variable into the readme.

This Pull Request (PR) fixes the following issues

<!--
Thank you for contributing to this project!

• This project has a Contributor Code of Conduct: https://voxpupuli.org/coc/
• Please check that here is no existing issue or PR that addresses your problem.
• Our vulnerabilities reporting process is at https://voxpupuli.org/security/

-->

Pull Request (PR) description

<!--
Replace this comment with a description of your pull request.
-->

This Pull Request (PR) fixes the following issues

<!--
Replace this comment with the list of issues or n/a.
Use format:
Fixes #123
Fixes #124
-->

The syslogfacility type is present in stdlib, so we should use it.
Before merging this,
https://github.com/puppetlabs/puppetlabs-stdlib/pull/1005 needs to be
released. We also need to update metadata.json

<!--
Thank you for contributing to this project!

• This project has a Contributor Code of Conduct: https://voxpupuli.org/coc/
• Please check that here is no existing issue or PR that addresses your problem.
• Our vulnerabilities reporting process is at https://voxpupuli.org/security/

-->

Pull Request (PR) description

<!--
Replace this comment with a description of your pull request.
-->

This Pull Request (PR) fixes the following issues

<!--
Replace this comment with the list of issues or n/a.
Use format:
Fixes #123
Fixes #124
-->

Pull Request (PR) description

Converted the project work with PDK 2.5.0
Fixed merge behavior when nrpe::commands or nrpe::plugins resources gets created at different hiera levels
i.e. OS.yaml and node.yaml

This Pull Request (PR) fixes the following issues

Fixes #67

<!--
Thank you for contributing to this project!

• This project has a Contributor Code of Conduct: https://voxpupuli.org/coc/
• Please check that here is no existing issue or PR that addresses your problem.
• Our vulnerabilities reporting process is at https://voxpupuli.org/security/

-->

Pull Request (PR) description

I want to be able to configure ssl protocol settings even if I don't use ssl cert authentication.
SSL directive can be included all time even if daemon is started with no-ssl flag.

This Pull Request (PR) fixes the following issues

<!--
Replace this comment with the list of issues or n/a.
Use format:
Fixes #123
Fixes #124
-->

Pull Request (PR) description

Add FreeBSD Support gt 12.3 and 13.1.
Since FreeBSD 12.4 and FreeBSD 13.2, also FreeBSD 14 they renamed the nrpe3 package and servicename into nrpe (nrpe-4x).
In that case this module will fail if you do not change manage_package to false or set package_name and service_name to nrpe.
With this PR the module will work without todos like this.

This Pull Request (PR) fixes the following issues

n/a

BR. Toutzn