Refactor package installation

ansible/roles/appliance/tasks/dependencies.yml

@@ -31,7 +31,7 @@
 
 - name: Install distro packages
   ansible.builtin.apt:
-    pkg: "{{ dependencies.packages }}"
+    pkg: "{{ dependencies.packages.apt }}"
     state: latest
     update_cache: yes
     install_recommends: no
@@ -42,6 +42,11 @@
     state: latest
     update_cache: yes
 
+- name: Install additional python packages
+  ansible.builtin.pip:
+    name: "{{ dependencies.packages.pip }}"
+    state: latest
+
 - name: Cleanup apt cache
   ansible.builtin.apt:
     autoremove: yes

ansible/roles/os/tasks/ansible.yml

@@ -1,6 +0,0 @@
-- name: Install ansible (w/ dependencies)
-  ansible.builtin.pip:
-    name: "{{ item }}"
-    executable: pip3
-    state: latest
-  loop: "{{ pip_packages }}"

ansible/roles/os/tasks/main.yml

@@ -15,6 +15,3 @@
 
 - name: Install packages
   import_tasks: packages.yml
-
-- name: Install ansible
-  import_tasks: ansible.yml

ansible/roles/os/tasks/packages.yml

@@ -13,8 +13,10 @@
 
 - name: Install additional python packages
   ansible.builtin.pip:
-    name: "{{ packages.pip }}"
+    name: "{{ item }}"
+    executable: pip3
     state: latest
+  loop: "{{ packages.pip }}"
 
 - name: Cleanup
   ansible.builtin.apt:

ansible/roles/os/vars/main.yml

@@ -1,11 +1,13 @@
 packages:
-  - jq
-  # (python3-*) Dependency for installation of Ansible
-  - python3-pip
-  - python3-setuptools
-  - python3-wheel
-
-pip_packages:
-  - pip
-  - ansible-core
-  - lxml
+  apt:
+    - jq
+    # (python3-*) Dependency for installation of Ansible
+    - python3-pip
+    # - python3-setuptools
+    # - python3-wheel
+  pip:
+    - ansible-core
+    - lxml
+    - pip
+    - setuptools
+    - wheel