Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add day, month and day of week option to cron job. #12

Merged
merged 1 commit into from
Jul 13, 2022
Merged

Add day, month and day of week option to cron job. #12

merged 1 commit into from
Jul 13, 2022

Conversation

bschonec
Copy link
Contributor

Added day of month, month and day of week options to the AIDE cron job. In my environment, running the AIDE job every day was taxing my servers too much. Once a week or once a month is sufficient for my purposes.

Also, I simplified the cron.pp manifest a bit since all that logic was just to build the command variable for the cron job.

I have no idea how to update spec tests and am trying to learn how to do so. If someone can help add spec tests, I would be most appreciative.

@kekogya
Copy link
Contributor

kekogya commented Jul 13, 2022

@bschonec, thank you for your contribution to this code base. I will merge your change and then work on updating the module on puppet forge in the next couple of days.

@kekogya kekogya merged commit 70a664b into indiana-university:master Jul 13, 2022
@kekogya kekogya added the enhancement New feature or request label Jul 13, 2022
@bschonec bschonec deleted the add_cron_day_month_dayofweek branch July 14, 2022 15:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants