Correctly name workers.ini, move worker restart to handler

handlers/main.yml

@@ -1,2 +1,15 @@
 ---
-# Handlers
+- name: Reload firewalld
+  ansible.builtin.systemd:
+    name: firewalld
+    state: reloaded
+  ignore_errors: "{{ ansible_check_mode }}"
+
+- name: Restart openQA workers
+  ansible.builtin.systemd:
+    name: "openqa-worker@{{ item }}"
+    state: restarted
+    enabled: true
+  # range "end" parameter is exclusive, so add 1
+  loop: "{{ range(1, (openqa_worker_count | int + 1)) | list }}"
+  ignore_errors: "{{ ansible_check_mode }}"

tasks/openqa-worker.yml

@@ -30,14 +30,7 @@
     - openqa-vnc
   tags:
     - configure
-
+  notify: Reload firewalld
-- name: Reload firewalld
-  ansible.builtin.systemd:
-    name: firewalld
-    state: reloaded
-  tags:
-    - configure
-  ignore_errors: "{{ ansible_check_mode }}"
 
 - name: Write openQA configuration file
   ansible.builtin.template:
@@ -48,19 +41,17 @@
     mode: "0444"
   loop:
     - client.conf
-    - workers.conf
+    - workers.ini
   tags:
     - configure
+  notify: Restart openQA workers
 
-- name: Start openQA workers
+- name: Start openQA cache services
   ansible.builtin.systemd:
-    name: "openqa-worker@{{ item }}"
+    name: "{{ item }}"
     state: started
     enabled: true
-  # range "end" parameter is exclusive, so add 1
+  loop:
-  loop: "{{ range(1, (openqa_worker_count | int + 1)) | list }}"
+    - openqa-worker-cacheservice
-  tags:
+    - openqa-worker-cacheservice-minion
-    - start_workers
-    - configure
-  ignore_errors: "{{ ansible_check_mode }}"
 ...