Adding cron recipe to setup regular backups #3
Conversation
Adding cron dependency Added cron documentation Updated docs doc update Link fix Metadata
|
Hi @brint, thanks for the PR! I actually left the CRON-recipe out of the latest version of the cookbook as I thought it would be out of the scope for this cookbook... But if there's a need to get it back in, I don't mind either :-) Kind regards, |
|
Hey @djoos, You can't +1 your own PR can you? :) Thanks for maintaining the cookbook! I do think cron is useful inside the cookbook. It's completely isolated from everything else, so the recipe would have to be called directly to setup cron. It's something we'll be using for sure, and keeps us from having to write and maintain additional site cookbooks. Let me know what you think! Brint |
|
Added a commit to add RHEL/CentOS support as well. |
|
Hi @brint, sorry for the huge delay in getting back to you! A recent PR (#4) added in CentOS/RHEL support... I'm actually still not 100% sure whether the CRON-support belongs in the cookbook rather than in a wrapper cookbook, hence why it hasn't been merged yet. Do let me know your thoughts so we can make progress with this PR. Thanks in advance for your feedback! Kind regards, |
PR Includes:
hollandbackup::cron)README.mdformatting