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

Fix warnings concerning deprecated use of bare variables #96

Merged
merged 1 commit into from
Apr 29, 2016

Conversation

pchaigno
Copy link
Collaborator

Using bare variable in with_items results in the following with the last version of Ansible (2.0.2.0-1):

[DEPRECATION WARNING]: Using bare variables is deprecated. Update your playbooks
 so that the environment value uses the full variable syntax ('{{some_var}}').
This feature will be removed in a future release. Deprecation warnings can be
disabled by setting deprecation_warnings=False in ansible.cfg.

Using bare variable in with_items results in the following with the last version of Ansible:
[DEPRECATION WARNING]: Using bare variables is deprecated. Update your playbooks
 so that the environment value uses the full variable syntax ('{{some_var}}').
This feature will be removed in a future release. Deprecation warnings can be
disabled by setting deprecation_warnings=False in ansible.cfg.
@pchaigno pchaigno added the bug label Apr 29, 2016
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.09%) to 87.252% when pulling 5999b0c on warning-variables into 3838a44 on master.

@awailly
Copy link
Owner

awailly commented Apr 29, 2016

I kind of preferred the with_items: on several lines, but if this is how they changed the format, good to merge!

@awailly awailly merged commit bee93d0 into master Apr 29, 2016
@awailly awailly deleted the warning-variables branch April 29, 2016 20:30
@pchaigno
Copy link
Collaborator Author

I put all with_items on a single line for consistency, both across with_items statements and with regard to other single-value statements.

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

Successfully merging this pull request may close these issues.

None yet

3 participants