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

Various deprecation warnings. #1115

Closed
4 tasks done
nextgenthemes opened this issue Oct 13, 2019 · 1 comment · Fixed by #1175
Closed
4 tasks done

Various deprecation warnings. #1115

nextgenthemes opened this issue Oct 13, 2019 · 1 comment · Fixed by #1175

Comments

@nextgenthemes
Copy link
Contributor

Description

TASK [letsencrypt : Get list of hosts in current Nginx conf] ****************************************
[DEPRECATION WARNING]: evaluating ssl_enabled and item.value.ssl.provider | default('manual') == 
'letsencrypt' as a bare variable, this behaviour will go away and you might need to add |bool to the
 expression in the future. Also see CONDITIONAL_BARE_VARS configuration toggle.. This feature will 
be removed in version 2.12. Deprecation warnings can be disabled by setting 
deprecation_warnings=False in ansible.cfg.
ok: [116.203.226.1] => (item=nextgenthemes.com)

TASK [letsencrypt : Create needed Nginx confs for challenges] ***************************************
[DEPRECATION WARNING]: evaluating ssl_enabled and item.value.ssl.provider | default('manual') == 
'letsencrypt' as a bare variable, this behaviour will go away and you might need to add |bool to the
 expression in the future. Also see CONDITIONAL_BARE_VARS configuration toggle.. This feature will 
be removed in version 2.12. Deprecation warnings can be disabled by setting 
deprecation_warnings=False in ansible.cfg.
skipping: [116.203.226.1] => (item=nextgenthemes.com) 

TASK [letsencrypt : Enable Nginx sites] *************************************************************
[DEPRECATION WARNING]: evaluating ssl_enabled and item.value.ssl.provider | default('manual') == 
'letsencrypt' as a bare variable, this behaviour will go away and you might need to add |bool to the
 expression in the future. Also see CONDITIONAL_BARE_VARS configuration toggle.. This feature will 
be removed in version 2.12. Deprecation warnings can be disabled by setting 
deprecation_warnings=False in ansible.cfg.

...

TASK [wordpress-setup : Warn about deprecated Nginx includes directory] *****************************
[DEPRECATION WARNING]: Using tests as filters is deprecated. Instead of using `result|search` use 
`result is search`. This feature will be removed in version 2.9. Deprecation warnings can be 
disabled by setting deprecation_warnings=False in ansible.cfg.

There is actually more in there I did not copy anything, and its seems most, if not all come from trellis roles not 3rd party.

Steps to reproduce

  1. vagrant provison or server playbook.

Expected behavior:

No warnings.

Actual behavior:

deprecation warnings

Reproduces how often:

always

Versions

Suse Tubleweed ~2019-10-12
Ansible 2.8.3
Trellis master merged into my project yesterday 2019-10-12

Additional information

Just wanted to get you a heads up for this in case this is not already tracked somewhere.

@swalkinshaw
Copy link
Member

Thanks 👍 These might be easy to fix unless 2.7 doesn't support the replacement

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 a pull request may close this issue.

2 participants