voxpupuli/puppet-logrotate

Deprecated su_owner parameter referenced in README.md

Opened this issue · 2 comments

Affected Puppet, Ruby, OS and module versions/distributions

  • Module version: 3.4.0

What are you seeing

README.md still refers to deprecated su_owner parameter. If used as documented manifest compilation will fail:

Error: Could not retrieve catalog from remote server: Error 500 on SERVER: Server Error: Evaluation Error: Error while evaluating a Resource Statement, Logrotate::Rule[somerule]: has no parameter named 'su_owner'

What behaviour did you expect instead

README.md should contain correct information.

Hey @synaptis thank you for the report.

Can you propose a PR to fix this ? Or even better switch the module to REFERENCE.md docs ?

For instance this file https://github.com/voxpupuli/puppet-ferm/blob/master/REFERENCE.md
is produced from comments at top of pp files with a command like puppet strings generate --format markdown --out REFERENCE.md.

it's also referenced in a comment in rule.pp