-
-
Notifications
You must be signed in to change notification settings - Fork 2
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
standard cron #26
standard cron #26
Conversation
Signed-off-by: Eric Nemchik <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for opening this pull request! Be sure to follow the pull request template!
I am a bot, here are the test results for this PR:
|
Signed-off-by: Eric Nemchik <[email protected]>
Signed-off-by: Eric Nemchik <[email protected]>
I am a bot, here are the test results for this PR:
|
Signed-off-by: Eric Nemchik <[email protected]>
I am a bot, here are the test results for this PR:
|
This pull request has been automatically marked as stale because it has not had recent activity. This might be due to missing feedback from OP. It will be closed if no further activity occurs. Thank you for your contributions. |
I am a bot, here are the test results for this PR:
|
Base image changes:
/defaults/crontabs/abc
or/defaults/crontabs/root
exist (in the downstream image) setup/config/crontabs/
for the user and import user crontabs to systemabc
crontab unless absolutely necessary (ex: rsnapshot needs root, cron mod sets up both so the user can do whatever they want)These should be safe to merge anytime as they don't actually do anything unless the downstream image (or the cron mod) include crontabs, or technically the user can mount
/etc/crontabs/abc
or/etc/crontabs/root
to their image, but we should only ever ship/defaults/crontabs/*
in downstream images so we're consistent in setting up the/config/crontabs/*