Fix certbot reload

handlers/main.yml

@@ -1,7 +1,7 @@
 ---
 
-- name: Restart certbot
-  systemd:
+- name: Enable certbot
+  ansible.builtin.systemd:
     name: certbot-renew.timer
-    state: restarted
+    enabled: true
     daemon_reload: true

meta/main.yml

@@ -5,15 +5,14 @@ galaxy_info:
   company: ELAN e.V.
   description: Configure TLS certificate renewal via certbot for Opencast
   license: BSD-3-Clause
-  min_ansible_version: 2.9
+  min_ansible_version: "2.9"
   galaxy_tags:
     - opencast
   platforms:
     - name: EL
       versions:
-        - 7
-        - 8
-        - 9
+        - "8"
+        - "9"
     - name: Debian
       versions:
         - all

tasks/main.yml

@@ -8,15 +8,18 @@
   ansible.builtin.package:
     name: certbot
 
-- name: Install certbot services
-  ansible.builtin.template:
-    src: '{{ item }}'
-    dest: /etc/systemd/system/{{ item }}
-    mode: '0644'
+# This task should remove custom service definitions created by previous version of this role.
+# Certbot package come with a renew service definitions itself, why not using it?
+# It will renew all registred certificates by default and does not limit us on some domains.
+# You can remove this task on later versions.
+- name: Remove custom certbot services definition
+  ansible.builtin.file:
+    path: '/etc/systemd/system/{{ item }}'
+    state: absent
   loop:
     - certbot-renew.service
     - certbot-renew.timer
-  notify: Restart certbot
+  notify: Enable certbot
 
 - name: Start certbot service
   ansible.builtin.service: