move service task and handler to a separate role (see ansible issue 23389, 20603, 15902)

Resolves #13

roles/apache2/handlers/main.yaml

@@ -1,5 +0,0 @@
----
-- include: service.yaml
-  # static: yes # see static include issue: https://github.com/ansible/ansible/issues/13485
-  vars:
-    service_name: apache2

roles/apache2/tasks/main.yaml

@@ -1,5 +1,6 @@
 ---
-- include: service.yaml
+- include_role:
+    name: service
   # static: yes # see static include issue: https://github.com/ansible/ansible/issues/13485
   vars:
     service_name: apache2

roles/dovecot/handlers/main.yml

@@ -1,8 +0,0 @@
----
-- include: service.yaml
-  vars:
-    service_name: postfix
-
-- include: service.yaml
-  vars:
-    service_name: dovecot

roles/dovecot/tasks/main.yaml

@@ -1,4 +1,5 @@
-- include: service.yaml
+- include_role:
+    name: service
   # static: yes # see static include issue: https://github.com/ansible/ansible/issues/13485
   vars:
     service_name: dovecot

roles/exim4/handlers/main.yml

@@ -5,7 +5,3 @@
 - name: update-exim4.conf
   shell: /usr/sbin/update-exim4.conf
   notify: restart exim4
-
-- include: service.yaml
-  vars:
-    service_name: exim4

roles/exim4/tasks/main.yaml

@@ -53,7 +53,8 @@
     - restart exim4
 
 - block:
-  - include: service.yaml
+  - include_role:
+      name: service
     # static: yes # see static include issue: https://github.com/ansible/ansible/issues/13485
     vars:
       service_name: exim4

roles/fail2ban/handlers/main.yaml

@@ -1,5 +0,0 @@
----
-- include: service.yaml
-  # static: yes # see static include issue: https://github.com/ansible/ansible/issues/13485
-  vars:
-    service_name: fail2ban

roles/fail2ban/tasks/main.yaml

@@ -1,5 +1,6 @@
 ---
-- include: service.yaml
+- include_role:
+    name: service
   # static: yes # see static include issue: https://github.com/ansible/ansible/issues/13485
   vars:
     service_name: fail2ban

roles/ldap/handlers/main.yaml

@@ -1,4 +0,0 @@
----
-- include: service.yaml
-  vars:
-    service_name: slapd

roles/ldap/tasks/main.yaml

@@ -37,7 +37,8 @@
       question: 'phamm-ldap/init_base_ldap'
       vtype: 'boolean'
       value: false
-- include: service.yaml
+- include_role:
+    name: service
   vars:
     service_name: slapd
     service_packages:

roles/mysql/handlers/main.yaml

@@ -1,4 +0,0 @@
----
-- include: service.yaml
-  vars:
-    service_name: mysql

roles/mysql/tasks/main.yaml

@@ -1,5 +1,6 @@
 ---
-- include: service.yaml
+- include_role:
+    name: service
   vars:
     service_name: mysql
     service_packages:

roles/nginx/handlers/main.yaml

@@ -1,11 +1,4 @@
 ---
-- include: service.yaml
-  # static: yes # see static include issue: https://github.com/ansible/ansible/issues/13485
-  vars:
-    service_name: nginx
-    service_packages:
-      - nginx
-
 - name: validate nginx configuration
   command: nginx -t -c /etc/nginx/nginx.conf
   changed_when: False

roles/nginx/tasks/main.yaml

@@ -1,5 +1,6 @@
 ---
-- include: service.yaml
+- include_role:
+    name: service
   # static: yes # see static include issue: https://github.com/ansible/ansible/issues/13485
   vars:
     service_name: nginx

roles/pam-ldap/handlers/main.yaml

@@ -8,7 +8,3 @@
 - name: update libnss-ldap configuration
   shell: rm /etc/libnss_ldap.conf || true; dpkg-reconfigure libnss-ldap -f noninteractive
   notify: restart nscd
-
-- include: service.yaml
-  vars:
-    service_name: nscd

roles/postfix/handlers/main.yml

@@ -1,13 +0,0 @@
----
-- include: service.yaml
-  vars:
-    service_name: postfix
-
-
-- include: service.yaml
-  vars:
-    service_name: clamav-daemon
-
-- include: service.yaml
-  vars:
-    service_name: amavisd-milter

roles/postfix/tasks/antispam.yaml

@@ -1,11 +1,13 @@
-- include: service.yaml
+- include_role:
+    name: service
   vars:
     service_name: clamav-daemon
     service_packages:
            - clamav-daemon
     install_recommends: yes
 
-- include: service.yaml
+- include_role:
+    name: service
   vars:
     service_name: amavisd-milter
     service_packages:
@@ -41,7 +43,8 @@
     regexp: '^non_smtpd_milters='
   notify: restart postfix
 
-- include: service.yaml
+- include_role:
+    name: service
   vars:
     service_name: postgrey
     service_packages:

roles/postfix/tasks/main.yaml

@@ -11,7 +11,8 @@
       vtype: 'string'
       value: 'lilik.it'
 
-- include: service.yaml
+- include_role:
+    name: service
   vars:
     service_name: postfix
     service_packages:
@@ -20,7 +21,7 @@
       # TODO: log, add a centralized log server
       - rsyslog
 
-- name: configure ldap-aliases for postfix 
+- name: configure ldap-aliases for postfix
   template:
      src: ldap-aliases.cf.j2
      dest: "/etc/postfix/ldap-aliases.cf"

roles/postgresql/handlers/main.yaml

@@ -1,4 +0,0 @@
----
-- include: service.yaml
-  vars:
-    service_name: postgresql

roles/postgresql/tasks/main.yaml

@@ -1,6 +1,7 @@
 ---
 # include tasks from task/service.yaml
-- include: service.yaml
+- include_role:
+    name: service
   vars:
     service_name: postgresql
     service_packages:

roles/ssh_server/handlers/main.yaml

@@ -1,8 +1,4 @@
----
-- include: service.yaml
-  # static: yes # see static include issue: https://github.com/ansible/ansible/issues/13485
+# We can not use include_role here since it not share thje connection with the current role
+- include: roles/service/handlers/main.yaml
   vars:
     service_name: ssh
-    service_packages:
-      - openssh
-