cron_hour (and cron_minute I guess) should allow "commas"

tohuwabohu / puppet-duplicity

Puppet module to manage backups based on duplicity.

Apache License 2.0

8 stars 33 forks source link

cron_hour (and cron_minute I guess) should allow "commas" #14

Closed TJM closed 8 years ago

TJM commented 9 years ago

Puppet error:

Error: Parameter hour failed on Cron[backup-system]: 1,13 is not a valid hour at /etc/puppet/modules/duplicity/manifests/profile.pp:275

The "cron_hour" should have validation that allows for backup runs more than once per day by adjusting cron_hour.

TJM commented 8 years ago

It was pointed out to me (privately) that puppet-duplicity is not restricting the value, it is the "cron" type in puppet, which does clearly state that you need to use an "Array" to pass multiple values in such as [1,13] instead of "1,13" ... so I will close this request with that information in case anyone else runs into that.