From e29311c4ed3cc2f03507c3bc7f3bcf5c0ac92a70 Mon Sep 17 00:00:00 2001
From: Alexander Dahl <alex@netz39.de>
Date: Sat, 23 Jul 2022 22:15:39 +0200
Subject: [PATCH] Move tasks for all hosts to separate file

When working on !54 it became obvious playbooks could be structured
better in this project.  Therefor we move the tasks supposed to be run
for _all_ hosts to a separate file.  The *main.yml* file should only
import other playbooks from now on, should not define tasks anymore.
---
 all.yml  | 38 ++++++++++++++++++++++++++++++++++++++
 main.yml | 37 +++----------------------------------
 2 files changed, 41 insertions(+), 34 deletions(-)
 create mode 100644 all.yml

diff --git a/all.yml b/all.yml
new file mode 100644
index 0000000..3d14478
--- /dev/null
+++ b/all.yml
@@ -0,0 +1,38 @@
+---
+# tasks for all hosts
+
+- hosts: all
+  become: true
+
+  vars:
+    ansible_python_interpreter: /usr/bin/python3
+
+  roles:
+    - role: ansible.timezone
+
+  tasks:
+    - name: Update and clean package cache
+      apt:
+        update_cache: true
+        cache_valid_time: 3600
+        autoclean: true
+      changed_when: false
+
+    - name: Ensure unattended-upgrades is installed and up to date
+      apt:
+        name: unattended-upgrades
+        state: present
+
+    - name: Setup unattended-upgrades
+      include_role:
+        name: ansible-role-unattended-upgrades
+      vars:
+        unattended_origins_patterns:
+          - "origin=Debian,archive=buster-security"
+          - "o=Debian,a=buster-updates"
+        unattended_package_blacklist: [cowsay]
+        unattended_mail: "root"
+
+    - name: Setup users
+      include_role:
+        name: users
diff --git a/main.yml b/main.yml
index 88c86ef..797b943 100644
--- a/main.yml
+++ b/main.yml
@@ -1,39 +1,8 @@
 ---
-- hosts: all
-  become: true
+# main playbook aggregating all the other playbooks
 
-  vars:
-    ansible_python_interpreter: /usr/bin/python3
-
-  roles:
-    - role: ansible.timezone
-
-  tasks:
-    - name: Update and clean package cache
-      apt:
-        update_cache: true
-        cache_valid_time: 3600
-        autoclean: true
-      changed_when: false
-
-    - name: Ensure unattended-upgrades is installed and up to date
-      apt:
-        name: unattended-upgrades
-        state: present
-
-    - name: Setup unattended-upgrades
-      include_role:
-        name: ansible-role-unattended-upgrades
-      vars:
-        unattended_origins_patterns:
-          - "origin=Debian,archive=buster-security"
-          - "o=Debian,a=buster-updates"
-        unattended_package_blacklist: [cowsay]
-        unattended_mail: "root"
-
-    - name: Setup users
-      include_role:
-        name: users
+- name: Setup for all hosts
+  import_playbook: all.yml
 
 - name: Specific setup for all proxmox hosts
   import_playbook: proxmox.yml