From 23f458517c51f79965df9ba27bc125618da5ad4f Mon Sep 17 00:00:00 2001 From: Arthur BOUDREAULT Date: Wed, 16 Feb 2022 13:57:59 +0100 Subject: [PATCH] refactor: add new pre-task to ynh_setup + fixes in README --- README-FR.md | 3 ++- README.md | 3 ++- roles/ynh_setup/README-FR.md | 6 ------ roles/ynh_setup/README.md | 6 ------ roles/ynh_setup/tasks/main.yml | 9 ++++++++- 5 files changed, 12 insertions(+), 15 deletions(-) diff --git a/README-FR.md b/README-FR.md index 1ad3f34..45d43b7 100644 --- a/README-FR.md +++ b/README-FR.md @@ -26,6 +26,7 @@ Votre serveur doit être basé sur du Debian Buster et Yunohost ne doit pas déj ### Rôles - [`lydra.yunohost.ynh_setup`](roles/ynh_setup/README-FR.md) : Ce rôle prépare les serveurs à base de Debian-Buster à exécuter Yunohost. Il configure Yunohost avec ses paramètres initiaux et les domaines, utilisateurs et applications de votre choix. +- [`lydra.yunohost.ynh_apps`](roles/ynh_apps/README-FR.md): Ce rôle installe les applications Yunohost de votre choix et peut également lancer des tâches de post-installation. - [`lydra.yunohost.ynh_config`](roles/ynh_config/README-FR.md) : Ce rôle gère la configuration de différents services de Yunohost (relais SMTP, mises à jour automatiques). - [`lydra.yunohost.ynh_backup`](roles/ynh_backup/README-FR.md) : Ce rôle gère la configuration des sauvegardes. @@ -37,7 +38,7 @@ Ces tags sont applicables suivant les rôles. |tags|commentaires| |----|-------| -|yunohost|Tâches spécifiques à Yunohost.| +|yunohost|Tâches spécifiques à Yunohost lui-même (installation ou configuration).| |users|Tâches spécifiques aux utilisateurs de Yunohost.| |domains|Tâches spécifiques aux domaines liés à Yunohost.| |apps|Tâches spécifiques aux applications de Yunohost.| diff --git a/README.md b/README.md index 239c7bc..03cc4a1 100644 --- a/README.md +++ b/README.md @@ -26,6 +26,7 @@ Your server must be Debian-Buster based and Yunohost shouldn't be already instal ### Roles - [`lydra.yunohost.ynh_setup`](roles/ynh_setup/README.md): This role prepares servers with Debian-Buster-based to run Yunohost. It sets up Yunohost with its initial settings and domains, users and apps of your choice. +- [`lydra.yunohost.ynh_apps`](roles/ynh_apps/README.md): This role installs Yunohost apps of your choice and can perform post-install tasks. - [`lydra.yunohost.ynh_config`](roles/ynh_config/README.md): This role configures various Yunohost services (SMTP relay, auto updates). - [`lydra.yunohost.ynh_backup`](roles/ynh_backup/README.md): This role manages the configuration of backups. @@ -37,7 +38,7 @@ These tags are applicable to roles. |tags|comment| |----|-------| -|yunohost|Tasks specific to Yunohost.| +|yunohost|Tasks specific to Yunohost itself (setup or configuration).| |users|Tasks specific to users in Yunohost.| |domains|Tasks specific to domains linked to Yunohost.| |apps|Tasks specific to Yunohost apps.| diff --git a/roles/ynh_setup/README-FR.md b/roles/ynh_setup/README-FR.md index 1d3687e..a3976de 100644 --- a/roles/ynh_setup/README-FR.md +++ b/roles/ynh_setup/README-FR.md @@ -79,12 +79,6 @@ Aucune. become: True collections: - lydra.yunohost - pre_tasks: - - name: Update all packages and index - ansible.builtin.apt: - upgrade: dist - update_cache: yes - roles: - ynh_setup - ynh_apps diff --git a/roles/ynh_setup/README.md b/roles/ynh_setup/README.md index 5b6afb2..60edfa2 100644 --- a/roles/ynh_setup/README.md +++ b/roles/ynh_setup/README.md @@ -79,12 +79,6 @@ None. become: True collections: - lydra.yunohost - pre_tasks: - - name: Update all packages and index - ansible.builtin.apt: - upgrade: dist - update_cache: yes - roles: - ynh_setup - ynh_apps diff --git a/roles/ynh_setup/tasks/main.yml b/roles/ynh_setup/tasks/main.yml index a2d22ba..4198159 100644 --- a/roles/ynh_setup/tasks/main.yml +++ b/roles/ynh_setup/tasks/main.yml @@ -17,6 +17,13 @@ # along with this program. If not, see . # # # #-----------------------------------------------------------------------------# +- name: Update all packages and index + ansible.builtin.apt: + upgrade: dist + update_cache: yes + tags: + - pkg + - linux - name: Install requirements ansible.builtin.apt: @@ -45,7 +52,7 @@ dest: "{{ item.link }}" state: link with_items: - - "{{ ynh_data_dirs }}" + - "{{ ynh_data_dirs }}" tags: - linux when: ynh_data_dirs_enabled