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

handful of small Ansible fixes #26

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

Conversation

afeld
Copy link

@afeld afeld commented Jan 25, 2017

See commits. Let me know if you'd like me to break them up into individual pull requests. Thanks!

Deprecated as of Ansible 2.2. Output:

```
TASK [silpion.tomcat : Register static/unmanaged conf files]
*******************
[DEPRECATION WARNING]: always_run is deprecated. Use check_mode = no
instead..
This feature will be
 removed in version 2.4. Deprecation warnings can be disabled by setting
deprecation_warnings=False
 in ansible.cfg.
```
From [the `template`
docs](http://docs.ansible.com/ansible/template_module.html#options):

> Leaving off the leading zero will likely have unexpected results.

which I confirmed:

```
$ ls -l /etc/systemd/system/tomcat@.service
--w----r-T. 1 root root 1678 Jan 24 23:17
/etc/systemd/system/tomcat@.service
```
@rbramwell
Copy link

Looks good to me. I was just about to submit a PR for similar changes to resolve the deprecation warnings. Merged the changes from @afeld into my copy of the repo.

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.

None yet

2 participants