From 472384d4039d10dcfe1567b07f64ca6b1f8b744d Mon Sep 17 00:00:00 2001
From: LUXEY Adrien <adrien.luxey@inria.fr>
Date: Sun, 14 Mar 2021 15:49:08 +0100
Subject: =?UTF-8?q?premier=20tri=20des=20outils=20de=20base=20=C3=A0=20ins?=
 =?UTF-8?q?taller=20via=20apt?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 os/config/cluster_nodes.yml           |  2 ++
 os/config/roles/common/tasks/main.yml | 43 +++++++++++++++++++----------------
 2 files changed, 26 insertions(+), 19 deletions(-)

diff --git a/os/config/cluster_nodes.yml b/os/config/cluster_nodes.yml
index ea58630..0f22baf 100644
--- a/os/config/cluster_nodes.yml
+++ b/os/config/cluster_nodes.yml
@@ -1,6 +1,8 @@
 ---
  
 - hosts: cluster_nodes
+  # "you can define how many hosts Ansible should manage at a single time 
+  # using the serial keyword"
   serial: 1
   roles:
     - role: common
diff --git a/os/config/roles/common/tasks/main.yml b/os/config/roles/common/tasks/main.yml
index b4d00bb..f31b2c3 100644
--- a/os/config/roles/common/tasks/main.yml
+++ b/os/config/roles/common/tasks/main.yml
@@ -15,30 +15,35 @@
 - name: "Install base tools"
   apt:
     name:
-      - vim
-      - htop
-      - screen
-      - iptables
-      - iptables-persistent
-      - nftables
-      - iproute2
+      # Essentials
       - curl
-      - iputils-ping
-      - dnsutils
+      - less
+      - sudo
+      - tar
+      - unzip
+      # User tooling
+      - screen
+      - vim
+      # Monitoring
       - bmon
+      - htop
       - iftop
+      - iputils-ping
       - iotop
-      - docker.io
-      - unzip
-      - tar
-      - tcpdump
-      - less
-      - parted
-      - btrfs-tools
-      - libnss-resolve
-      - net-tools
       - strace
-      - sudo
+      - tcpdump
+      # Networking
+      - iproute2 # advanced net-tools
+      - iptables # legacy firewall (still used by diplonat)
+      - iptables-persistent
+      - net-tools # basic network tools
+      - nftables # iptables' successor (will replace it eventually)
+      - dnsutils # now called bind9-dnsutils
+      # Optional / Dispensable
+      #- docker.io # Adrien n'approuve pas (il faut utiliser le repo Docker)
+      - parted 
+      #- btrfs-tools
+      #- libnss-resolve # provides DNS/LLMNR utilities via systemd-resolved
     state: present
 
 - name: "Passwordless sudo"
-- 
cgit v1.2.3