From d4310781cc3ab8d10965b2644ee75423150084af Mon Sep 17 00:00:00 2001
From: Arthur BOUDREAULT <arthur@lydra.fr>
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 <http://www.gnu.org/licenses/>.        #
 #                                                                             #
 #-----------------------------------------------------------------------------#
+- 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
-- 
GitLab