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
Fix incorrect template data-type and add tests.
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
Add the ability to manage the user and group for nrpe. By default, this is turned off, but can easily be turned on if the user or group were not added automatically by the package.
Pull Request (PR) description
This PR splits the package_name and manage_package parameters into two extra pairs of parameters, one for the daemon and one for the plugins. This allows more granular control of the deployments. In the use case of "the puppet module controls the daemon package but not the plugins package", without this PR you must replicate much of the os-based choices of manifests/params.pp outside of the module in order to ensure that you get the right package names, when a much cleaner path is to simply capture that information in the module itself.
This is somewhat clumsy in that it introduces a duplication in the API, where you could end up with two different code paths controlling the same packages, so I'm quite open to changing patch 2 to be 'better' in terms of what's enabled/disabled, or if manage_package is changed to false in preparation for deprecation, or whatever seems most sane.
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!
-->
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
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
<!--
Thank you for contributing to this project!
-->
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
-->
<!--
Thank you for contributing to this project!
-->
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
-->
