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

cron module: Add mailto parameter #5466

Closed
wants to merge 1 commit into from

Conversation

RSpiertz
Copy link

This patch adds a mailto parameter to the cron module. It has been tested on Debian with both crontabs and separate cron files.

@AnneTheAgile
Copy link

1.Ticket #4584, adding env variables, is a more general version of this PR.
1a.This PR was requested to change its implementation to not hard-code.
2.I had not been aware that crontab could include environment setting lines! I can't find info on any implementations that don't support it but perhaps my Google-fu is not good.
http://man7.org/linux/man-pages/man5/crontab.5.html
2b.To avoid the extra line, of course one may export inside one's script.
http://stackoverflow.com/questions/2229825/where-can-i-set-environment-variables-that-crontab-will-use

@mpdehaan
Copy link
Contributor

Hi!

Thanks very much for your interest in Ansible. It sincerely means a lot to us.

On September 26, 2014, due to enormous levels of contribution to the project Ansible decided to reorganize module repos, making it easier
for developers to work on the project and for us to more easily manage new contributions and tickets.

We split modules from the main project off into two repos, http://github.com/ansible/ansible-modules-core and http://github.com/ansible/ansible-modules-extras

If you still would like this pull request merged, we will need your help making this target the new repo. If you do not take any action, this
pull request unfortunately cannot be applied.

We apologize that we are not able to make this transition happen seamlessly, though this is a one-time change and your help is greatly appreciated --
this will greatly improve velocity going forward.

Both sets of modules will ship with Ansible, though they'll receive slightly different ticket handling.

To locate where a module lives between 'core' and 'extras'

Otherwise, if this is a new module:

It may be possible to re-patriate your pull requests automatically, one user-submitted approach for advanced git users
has been suggested at https://gist.github.com/willthames/afbaaab0c9681ed45619

Additionally, should you need more help with this, you can ask questions on:

Thank you very much!

@AnneTheAgile
Copy link

This ticket should probably remain closed. On the new repo, I mentioned it at ;
ansible/ansible-modules-core#50

johnnynotsolucky pushed a commit to johnnynotsolucky/ansible that referenced this pull request Dec 30, 2016
@ansibot ansibot added feature This issue/PR relates to a feature request. and removed feature_pull_request labels Mar 4, 2018
@ansible ansible locked and limited conversation to collaborators Apr 24, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
feature This issue/PR relates to a feature request.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants