hosting/ynh-lydra-ansible-yunohost
1
0
Fork 0
Code Issues 2 Activity

♻refactor: using latest ansible module to be more future-proof and support latest ansible versions

Browse source 
Ansible-lint:
```
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)
This commit is contained in:
Arthur BOUDREAULT 2021-10-12 14:34:35 +00:00 committed by Christophe Chaudier
parent c30a2214a0
commit 39bd859070
1 changed files with 3 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
tasks/main.yml
View file

@ -30,7 +30,7 @@
when: ynh_file_install.stat.exists == False when: ynh_file_install.stat.exists == False
- name: Create extra domains - name: Create extra domains
ansible.builtin.include: domains.yml ansible.builtin.include_tasks: domains.yml
when: ynh_extra_domains when: ynh_extra_domains
- name: Run first Yunohost diagnosis - name: Run first Yunohost diagnosis
@ -41,9 +41,9 @@
changed_when: False changed_when: False
- name: Add Yunohost users - name: Add Yunohost users
ansible.builtin.include: users.yml ansible.builtin.include_tasks: users.yml
when: ynh_users when: ynh_users
- name: Install Yunohost apps - name: Install Yunohost apps
ansible.builtin.include: apps.yml ansible.builtin.include_tasks: apps.yml
when: ynh_apps when: ynh_apps