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

Adding env to allow cron jobs with environment variables #4584

Closed
wants to merge 3 commits into from

Conversation

djatlantic
Copy link

No description provided.

@mpdehaan
Copy link
Contributor

we typically do commas for list seperation and then we could do type=list in the argument spec.

Will that work?

Also, yes, please update this to 4 space indents.

Thanks!

@aiwilliams
Copy link

👍

@mpdehaan mpdehaan added the P3 label Mar 19, 2014
@mpdehaan mpdehaan added P4 and removed P3 labels Apr 17, 2014
@vpetersson
Copy link

+1

1 similar comment
@vcastellm
Copy link
Contributor

+1

@AnneTheAgile
Copy link

1.Is the merge conflict here because @djatlantic somehow did not git merge the fix for changing to 4 spaces ? //edit 2014-09-29Mon : No more conflict note appears here - I'm not sure if that is due to its being closed with the reorg of the git repo?
2.This ticket would include the desired MAIL_TO functionality of #5466 .
3.Is that the only issue barring approval?
3b.The doc could use an update - doublequotes should surround the two variables, and change to 'Create'. I could do that if we like the fix.

#creat a cron with env
- cron: name="djstellar_getmodels" job="/home/djstellar/bin/getmodels.sh > /dev/null 2>&1" env=MAILTO=ops@gmail.com,SHELL=/bin/bash hour=1 minute=9

//AnneTheAgile ; update; reorg, added [3], [3b]

@DavidWittman
Copy link
Contributor

👍 I need this.

@mdesjardins
Copy link

+1 I need this too!

@jimsmith
Copy link

+1 I need this also.

@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

I'm unclear if there is/not a merge conflict any more. Also as much upvoted, maybe it can be accepted on the new repo. I mentioned it here; ansible/ansible-modules-core#50

@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