handlers/main.yml

@@ -1,15 +1,2 @@
 ---
-- 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 }}"
+# Handlers

tasks/openqa-worker.yml

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