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

Unify cron command w/ variable, allowing user specifciation #27

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

eengstrom
Copy link
Contributor

builds on #25 (for testing) and #19 (simply to break changes into smaller chunks). Probably want to merge those first, in that order.

See rationale and comments in #26

fixes #26

The latest `dns-lexicon` won't install on that platform.
Defaults to `"{{ dehydrated_install_root }}/dehydrated --cron"`

There were subtle variations on the call to this scattered through the
code, and this way one may change the command, or embed into a pipe to
redirect output as needed.  For example:

- Don't give up on first failure:

```
dehydrated_renew_command: "{{ dehydrated_install_root }}/dehydrated 
--cron --keep-going"
```

- More complicated example to redirect output to `syslog`:

```
dehydrated_renew_command: "bash -c '({{ dehydrated_install_root 
}}/dehydrated -c -g | logger -t dehydrated -p local7.info) 2>&1 | logger 
-t dehydrated -p local7.error -s'"
```

... although the latter is somewhat contrived and one would probably be 
better served by using a `systemd` timer rather than a cron job, which 
would then do effectively the same thing.
@JKetelaar
Copy link

Can we get this merged?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Allow user/playbook to specify dehyraded cron call
2 participants