Pour tout problème contactez-nous par mail : support@froggit.fr | La FAQ :grey_question: | Rejoignez-nous sur le Chat :speech_balloon:

Skip to content
Snippets Groups Projects

:white_check_mark:Resolve "ansible include module is to be deprecated"

Merged Arthur BOUDREAULT requested to merge 14-ansible-include-module-is-to-be-deprecated into master
All threads resolved!
1 file
+ 3
3
Compare changes
  • Side-by-side
  • Inline
  • 3b5eb72d
    :recycle:Refactor: Ansible-lint: · 3b5eb72d
    Arthur BOUDREAULT authored
    ```
    Include has some unintuitive behaviours depending on if it is running in a static or dynamic in play or in playbook context, in an effort to clarify behaviours we are moving to a new set modules (ansible.builtin.include_tasks, ansible.builtin.include_role, ansible.builtin.import_playbook, ansible.builtin.import_tasks) that have well established and clear behaviours.
    This module will still be supported for some time but we are looking at deprecating it in the near future.
    ```
    
    Fix: I have decided to go for ansible include_tasks module because it is more versatile and on a par with this module (for more info about differences between new modules, see [here](https://docs.ansible.com/ansible/latest/collections/ansible/builtin/include_module.html)
+ 3
3
@@ -30,7 +30,7 @@
when: ynh_file_install.stat.exists == False
- name: Create extra domains
ansible.builtin.include: domains.yml
ansible.builtin.include_tasks: domains.yml
when: ynh_extra_domains
- name: Run first Yunohost diagnosis
@@ -41,9 +41,9 @@
changed_when: False
- name: Add Yunohost users
ansible.builtin.include: users.yml
ansible.builtin.include_tasks: users.yml
when: ynh_users
- name: Install Yunohost apps
ansible.builtin.include: apps.yml
ansible.builtin.include_tasks: apps.yml
when: ynh_apps
Loading