Split user/logging tasks to separate task files (#22)

* Split user/logging tasks to separate task files

* Remove root package install from makefile

Makefile

@@ -15,7 +15,7 @@ test: lint
 
 poetry:
 	@type poetry >/dev/null || pip3 install poetry
-	@poetry install
+	@poetry install --no-root
 
 lint: poetry
 	poetry run yamllint .

tasks/logging.yml

@@ -0,0 +1,10 @@
+---
+- name: Configure logging
+  ansible.builtin.copy:
+    dest: "{{ docker_log_conf_file }}"
+    content: "{{ docker_log_conf | to_nice_json(indent=2) }}"
+    group: root
+    owner: root
+    mode: 0644
+  notify: restart docker service
+  when: docker_log_conf_manage

tasks/main.yml

@@ -132,27 +132,8 @@
         - name: Include service tasks
           ansible.builtin.include_tasks: service.yml
 
-    - name: Configure logging
-      ansible.builtin.copy:
-        dest: "{{ docker_log_conf_file }}"
-        content: "{{ docker_log_conf | to_nice_json(indent=2) }}"
-        group: root
-        owner: root
-        mode: 0644
-      notify: restart docker service
-      when: docker_log_conf_manage
-      tags: configure
-
-    - name: Manage docker groups
-      ansible.builtin.group:
-        name: "{{ docker_group }}"
-      when: docker_group_manage
-      tags: configure
-
-    - name: Manage docker users
-      ansible.builtin.user:
-        append: true
-        groups: "{{ docker_group }}"
-        name: "{{ docker_user }}"
-      when: docker_user_manage
-      tags: configure
+- name: Include logging tasks
+  ansible.builtin.include_tasks: logging.yml
+
+- name: Include user tasks
+  ansible.builtin.include_tasks: users.yml

tasks/users.yml

@@ -0,0 +1,12 @@
+---
+- name: Manage docker groups
+  ansible.builtin.group:
+    name: "{{ docker_group }}"
+  when: docker_group_manage
+
+- name: Manage docker users
+  ansible.builtin.user:
+    append: true
+    groups: "{{ docker_group }}"
+    name: "{{ docker_user }}"
+  when: docker_user_manage