From 34b03f4fa7786148f788e312315813dee7799be8 Mon Sep 17 00:00:00 2001
From: Ahmed Abdelhalim <pm@a14m.dev>
Date: Mon, 11 Aug 2025 00:32:03 +0200
Subject: Refactor to simplify debian network role

---
 roles/network/tasks/install-archlinux.yml |  8 --------
 roles/network/tasks/install-debian.yml    | 30 ++++++++++++++++++++++++------
 roles/network/tasks/main.yml              | 12 +-----------
 3 files changed, 25 insertions(+), 25 deletions(-)

(limited to 'roles/network/tasks')

diff --git a/roles/network/tasks/install-archlinux.yml b/roles/network/tasks/install-archlinux.yml
index be4cc5e7..788febea 100644
--- a/roles/network/tasks/install-archlinux.yml
+++ b/roles/network/tasks/install-archlinux.yml
@@ -6,11 +6,3 @@
     state: "present"
     update_cache: true
   tags: [required_for_boot]
-
-- name: "Ensrue network_packages_removed are removed"
-  become: true
-  community.general.pacman:
-    name: "{{ network_packages_removed }}"
-    state: "absent"
-    extra_args: "-n" # Removes config files too
-  # Not required for boot
diff --git a/roles/network/tasks/install-debian.yml b/roles/network/tasks/install-debian.yml
index f09a240c..7263bc84 100644
--- a/roles/network/tasks/install-debian.yml
+++ b/roles/network/tasks/install-debian.yml
@@ -7,10 +7,28 @@
     update_cache: true
   tags: [required_for_boot]
 
-- name: "Ensrue network_packages_removed are removed"
+- name: "Set network_default_services facts"
+  ansible.builtin.set_fact:
+    network_default_services:
+      - "dhcpcd"
+      - "networking"
+      - "wpa_supplicant"
+  tags: [required_for_boot]
+
+- name: "Disable conflicting services"
   become: true
-  ansible.builtin.apt:
-    name: "{{ network_packages_removed }}"
-    state: "absent"
-    purge: true
-  # Not required for boot
+  ansible.builtin.systemd_service:
+    name: "{{ item }}"
+    enabled: false
+    state: "stopped"
+  with_items: "{{ network_default_services }}"
+  when: not ansible_is_chroot
+  tags: [required_for_boot]
+
+- name: "(chroot): Disable conflicting services"
+  ansible.builtin.command:
+    cmd: "systemctl disable {{ network_default_services | join(' ') }}"
+  changed_when: true
+  when: ansible_is_chroot
+  # noqa: command-instead-of-module module doesn't work inside chroot
+  tags: [required_for_boot]
diff --git a/roles/network/tasks/main.yml b/roles/network/tasks/main.yml
index 44a9e300..e8bca0be 100644
--- a/roles/network/tasks/main.yml
+++ b/roles/network/tasks/main.yml
@@ -40,21 +40,11 @@
   # noqa: command-instead-of-module module doesn't work inside chroot
   tags: [required_for_boot]
 
-- name: "Ensure network_services are enabled"
-  become: true
-  ansible.builtin.systemd_service:
-    name: "{{ item }}"
-    enabled: true
-    state: "started"
-  with_items: "{{ network_services }}"
-  when: not ansible_is_chroot
-  # Not required for boot
-
 - name: "Configure NetworkManager"
   become: true
   tags: [required_for_boot]
   notify:
-    - "Restart NetworkManager"
+    - "Restart network services"
     - "Reload systemd"
   block:
     - name: "Configure NetworkManager"
-- 
cgit v1.2.3