From 9cc82874a1b45ab2e75f38096c9f37c7d5a406f3 Mon Sep 17 00:00:00 2001
From: Harald Pfeiffer <coding@lirion.de>
Date: Sun, 14 Apr 2024 21:30:38 +0200
Subject: Code fixes

---
 patch.yaml | 58 ++++++++++++++++++++++++++++++++--------------------------
 1 file changed, 32 insertions(+), 26 deletions(-)

(limited to 'patch.yaml')

diff --git a/patch.yaml b/patch.yaml
index 3ab323d..2ed2ea5 100644
--- a/patch.yaml
+++ b/patch.yaml
@@ -1,51 +1,57 @@
 ---
 # You may want to change the default to your favourite host (group) you run this on the most.
-- hosts: "{{ rthosts | default('CHANGE_ME') }}"
-  order: inventory
+- name: Arrange Inventory
+  hosts: "{{ rthosts | default('CHANGE_ME') }}"
+  # Change: "inventory" now became "default". "inventory" is being frowned upon by ansible-lint.
+  # (At least until "default" behaviour changes again in the future, I assume :-) )
+  order: default
   gather_facts: false
   # default: all in first step, but that shit requires (int)
   serial: 666
   tasks:
-  - name: Gather necessary facts
-    ansible.builtin.setup:
-      filter: "ansible_distribution*"
-  - name: Group hosts by distribution file variety
-    group_by:
-      # We choose to lowercase anything here as there should be no collisions but
-      # SUSE could be "SuSE" or "SUSE" (assumed and unverified, but you never know...)
-      key: "adfv_{{ ansible_distribution_file_variety | lower | default('none') }}"
+    - name: Gather necessary facts
+      ansible.builtin.setup:
+        filter: "ansible_distribution*"
+    - name: Group hosts by distribution file variety
+      ansible.builtin.group_by:
+        # We choose to lowercase anything here as there should be no collisions but
+        # SUSE could be "SuSE" or "SUSE" (assumed and unverified, but you never know...)
+        key: "adfv_{{ ansible_distribution_file_variety | lower | default('none') }}"
   tags:
-    - all
-- hosts: adfv_debian
-  order: inventory
+    - always
+- name: Trigger Debian patching role on Debian hosts
+  hosts: adfv_debian
+  order: default
   gather_facts: false
   # default: all in first step, but that shit requires (int)
   serial: 666
   tasks:
-  - name: Debian Patches
-    ansible.builtin.import_role:
-      name: "patch_debian"
+    - name: Debian Patches
+      ansible.builtin.import_role:
+        name: "patch_debian"
   tags:
     - debian
-- hosts: adfv_redhat
-  order: inventory
+- name: Trigger Red Hat patching role on Red Hat hosts
+  hosts: adfv_redhat
+  order: default
   gather_facts: false
   # default: all in first step, but that shit requires (int)
   serial: 666
   tasks:
-  - name: Red Hat Patches
-    ansible.builtin.import_role:
-      name: "patch_redhat"
+    - name: Red Hat Patches
+      ansible.builtin.import_role:
+        name: "patch_redhat"
   tags:
     - redhat
-- hosts: adfv_suse
-  order: inventory
+- name: Trigger SUSE patching role on SUSE hosts
+  hosts: adfv_suse
+  order: default
   gather_facts: false
   # default: all in first step, but that shit requires (int)
   serial: 666
   tasks:
-  - name: SUSE Patches
-    ansible.builtin.import_role:
-      name: "patch_suse"
+    - name: SUSE Patches
+      ansible.builtin.import_role:
+        name: "patch_suse"
   tags:
     - suse
-- 
cgit v1.2.3