Removed the !defined logic as the variable su_user and su_group are d…
…efined always, they default to undef which is a defined value. Moved to using a selector to test for undef and assign a fixed user or group, else use the values passed to the class
Corrected logic which wrongly used an OR, corrected the test which lo…
…oked at the parameter values not the internal variables. Protected the whole thing with the missing su parameter
Changed parameter from $su_owner to $su_user to align with conf.pp an…
…d the terminology used in logrotate man page.
Changed parameter from $su_owner to $su_user to align with conf.pp an…
…d the terminology used in logrotate man page.
Merge pull request #123 from voxpupuli/modulesync
modulesync 1.9.3
Allow cron to always mail its output.
I was used to the fact that redhat always mails its output. I enabled the managed_cron bool, and added the --verbose startup argument (for debugging) and all mail went away...
Merge pull request #125 from voxpupuli/modulesync
modulesync 1.9.4
Allow cron to always mail its output.
I was used to the fact that redhat always mails its output. I enabled the managed_cron bool, and added the --verbose startup argument (for debugging) and all mail went away...
update: Enforce minimal string length on logrotate arguments.
Introduced in PR #102
Removed the !defined logic as the variable su_user and su_group are d…
…efined always, they default to undef which is a defined value. Moved to using a selector to test for undef and assign a fixed user or group, else use the values passed to the class (Reworked a bit)
Merge pull request #127 from Heidistein/args2
Enforce minimal string length on logrotate arguments.
Better testing if su is required.
I have made the su boolean no longer optional, defaulting to false. I have made su_user and su_group no longer optional, defaulting to root. Previously this was set top optional/undef and lateron in the code it was changed to a fixed string when if was undef anyway. I have added the su boolean to the logrotate::config to be consistent.
Merge pull request #124 from Heidistein/master
Allow cron to always mail its output.
Merge pull request #128 from Heidistein/pr80fix2
Rebase(2) of Pull Request #80
Merge pull request #130 from voxpupuli/modulesync
modulesync 1.9.6
Merge pull request #132 from voxpupuli/stdlib
allow puppetlabs/stdlib 5.x
Merge pull request #135 from bastelfreak/acceptancetests
enable acceptance tests centos 6/7 ubuntu 14/16
Merge pull request #134 from rj667/add_support_debian
add support for Debian
Modified params for ubuntu to explicitly set not only the su_user and…
… su_group but also the su parameters
Merge pull request #138 from acurus-puppetmaster/fix/su_ubuntu
Fixed params for Ubuntu to ensure su is set