-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
docs: Improve cron tasks and fix link in documentation #2708
docs: Improve cron tasks and fix link in documentation #2708
Conversation
adding an example with cron inside container to avoid possible errors adding 'MAILTO=""' option to the cron tasks to prevent cron from sending messages to email after completing tasks
Co-authored-by: Brennan Kinney <5098581+polarathene@users.noreply.github.com>
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.
LGTM 👍
You might want to still confirm the request from @casperklein on the associated issue.
While I don't mind the MAILTO
mention since the comment makes it clear what it prevents, some users may prefer to still be notified about failures, while ignoring successful tasks.
adding '> /dev/null' instead of 'MAILTO=""' that only error messages come to the email 'postmaster@example.com'
Documentation preview for this PR is ready! 🎉 Built with commit: fbab83f |
@@ -184,23 +184,25 @@ The following configuration works nicely: | |||
# This assumes you're having `environment: ONE_DIR=1` in the env-mailserver, | |||
# with a consolidated config in `/var/mail-state` | |||
# | |||
# '> /dev/null' to send error notifications from 'stderr' to 'postmaster@example.com' |
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.
It's technically meant to omit stdout
to avoid reports that are unnecessary, thus only stderr
would trigger reports.
Description
Everything that was mentioned in the issue and fixing the link to the actual
Fixes #2702
Type of change
Checklist:
docs/
)