Refactor to simplify debian network role

3 files changed, 25 insertions, 25 deletions

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"