From e807a5b9e0bc3f2b96282d41807610656fa7874b Mon Sep 17 00:00:00 2001
From: Ahmed Abdelhalim <pm@a14m.dev>
Date: Thu, 12 Feb 2026 23:14:26 +0100
Subject: Add reboot to the playbook as post_task (controlled by flag)

---
 roles/ddcutil/handlers/main.yml                  | 5 +++++
 roles/ddcutil/tasks/main.yml                     | 2 ++
 roles/gnome/handlers/main.yml                    | 5 +++++
 roles/gnome/tasks/install-archlinux.yml          | 1 +
 roles/gnome/tasks/install-debian.yml             | 1 +
 roles/nvidia_gtx1060/handlers/main.yml           | 5 +++++
 roles/nvidia_gtx1060/tasks/install-archlinux.yml | 8 ++++++--
 7 files changed, 25 insertions(+), 2 deletions(-)
 create mode 100644 roles/gnome/handlers/main.yml

(limited to 'roles')

diff --git a/roles/ddcutil/handlers/main.yml b/roles/ddcutil/handlers/main.yml
index ed0ffa85..dfc50931 100644
--- a/roles/ddcutil/handlers/main.yml
+++ b/roles/ddcutil/handlers/main.yml
@@ -4,3 +4,8 @@
   ansible.builtin.command:
     cmd: "udevadm control --reload-rules"
   changed_when: true
+
+- name: "Require reboot"
+  ansible.builtin.set_fact:
+    # This flag is used to trigger a post_task of rebooting after the playbook finishes
+    reboot_required: true
diff --git a/roles/ddcutil/tasks/main.yml b/roles/ddcutil/tasks/main.yml
index ed4dc845..5b97b5e1 100644
--- a/roles/ddcutil/tasks/main.yml
+++ b/roles/ddcutil/tasks/main.yml
@@ -12,6 +12,7 @@
     mode: "0644"
     content: |
       i2c-dev
+  notify: "Require reboot"
 
 - name: "Ensure i2c group is created"
   become: true
@@ -35,3 +36,4 @@
       KERNEL=="i2c-[0-9]*", GROUP="i2c", MODE="0660"
   notify:
     - "Reload rules"
+    - "Require reboot"
diff --git a/roles/gnome/handlers/main.yml b/roles/gnome/handlers/main.yml
new file mode 100644
index 00000000..5b3adc50
--- /dev/null
+++ b/roles/gnome/handlers/main.yml
@@ -0,0 +1,5 @@
+---
+- name: "Require reboot"
+  ansible.builtin.set_fact:
+    # This flag is used to trigger a post_task of rebooting after the playbook finishes
+    reboot_required: true
diff --git a/roles/gnome/tasks/install-archlinux.yml b/roles/gnome/tasks/install-archlinux.yml
index 824f314e..5222ff3f 100644
--- a/roles/gnome/tasks/install-archlinux.yml
+++ b/roles/gnome/tasks/install-archlinux.yml
@@ -4,6 +4,7 @@
   ansible.builtin.package:
     name: "{{ gnome_packages }}"
     state: "present"
+  notify: "Require reboot"
 
 - name: "Enable GDM display manager"
   become: true
diff --git a/roles/gnome/tasks/install-debian.yml b/roles/gnome/tasks/install-debian.yml
index 34369156..9ecdecb6 100644
--- a/roles/gnome/tasks/install-debian.yml
+++ b/roles/gnome/tasks/install-debian.yml
@@ -5,6 +5,7 @@
     name: "{{ gnome_packages }}"
     state: "present"
     install_recommends: false
+  notify: "Require reboot"
   tags:
     # NOTE: the gnome-core packages resinstalls the avahi packages
     # Disabling the idemopotence test to prevent always reporting changed
diff --git a/roles/nvidia_gtx1060/handlers/main.yml b/roles/nvidia_gtx1060/handlers/main.yml
index 67408c79..56f4faec 100644
--- a/roles/nvidia_gtx1060/handlers/main.yml
+++ b/roles/nvidia_gtx1060/handlers/main.yml
@@ -4,3 +4,8 @@
   ansible.builtin.command:
     cmd: "mkinitcpio -P"
   changed_when: true
+
+- name: "Require reboot"
+  ansible.builtin.set_fact:
+    # This flag is used to trigger a post_task of rebooting after the playbook finishes
+    reboot_required: true
diff --git a/roles/nvidia_gtx1060/tasks/install-archlinux.yml b/roles/nvidia_gtx1060/tasks/install-archlinux.yml
index 0a8565ea..8d5134bb 100644
--- a/roles/nvidia_gtx1060/tasks/install-archlinux.yml
+++ b/roles/nvidia_gtx1060/tasks/install-archlinux.yml
@@ -19,7 +19,9 @@
     src: "nvidia.conf.j2"
     dest: "/etc/modprobe.d/nvidia.conf"
     mode: "0644"
-  notify: "Rebuild initramfs"
+  notify:
+    - "Rebuild initramfs"
+    - "Require reboot"
   tags:
     # Since the /etc/mkinitcpio isn't available on the docker container
     # Ignore this for testing,
@@ -47,7 +49,9 @@
     path: "/etc/mkinitcpio.conf"
     regexp: "^MODULES=\\(.*\\)"
     line: "MODULES=({{ all_modules | join(' ') }})"
-  notify: "Rebuild initramfs"
+  notify:
+    - "Rebuild initramfs"
+    - "Require reboot"
   tags:
     # Since the /etc/mkinitcpio isn't available on the docker container
     # Ignore this for testing,
-- 
cgit v1.2.3