Merge pull request ANXS#299 from jlozadad/jlozadad_handlers

added the use of handlers

handlers/main.yml

@@ -1,6 +1,15 @@
 # file: postgresql/handlers/main.yml
 
-- name: restart postgresql
-  service:
-    name: "{{ postgresql_service_name }}"
-    state: restarted
+  - name: restart postgresql with service
+    service:
+      name: "{{ postgresql_service_name }}"
+      state: restarted
+      enabled: yes
+
+  - name: restart postgresql with systemd
+    systemd:
+      name: "{{ postgresql_service_name }}"
+      state: restarted
+      enabled: yes
+    when: ansible_service_mgr == 'systemd'
+

tasks/configure.yml

@@ -136,6 +136,7 @@
     state: directory
     mode: 0755
   when: ansible_os_family == "RedHat"
+  notify: restart postgresql with systemd
 
 - name: PostgreSQL | Use the conf directory when starting the Postgres service | RedHat
   template:
@@ -151,15 +152,5 @@
     owner: "{{ postgresql_service_user }}"
     group: "{{ postgresql_service_group }}"
     mode: u=rwX,g=rwXs,o=rx
+  notify: restart postgresql with service
 
-- name: PostgreSQL | Enable service
-  service:
-    name: "{{ postgresql_service_name }}"
-    enabled: yes
-  when: postgresql_service_enabled
-
-- name: PostgreSQL | Restart PostgreSQL
-  service:
-    name: "{{ postgresql_service_name }}"
-    state: restarted
-  when: postgresql_configuration_pt1.changed or postgresql_configuration_pt2.changed or postgresql_configuration_pt3.changed or postgresql_systemd_custom_conf.changed

tasks/install_yum.yml

@@ -2,36 +2,39 @@
 
 # The standard ca-certs are needed because  without them apt_key will fail to
 # validate www.postgresql.org (or probably any other source).
-- name: PostgreSQL | Make sure the CA certificates are available
-  yum:
-    name: ca-certificates
-    state: present
 
-- name: PostgreSQL | Add PostgreSQL repository
-  yum:
-    name: "{{ postgresql_yum_repository_url }}"
-    state: present
+  - block:
+      - name: PostgreSQL | Make sure the CA certificates are available
+        yum:
+          name: ca-certificates
+          state: present
 
-- name: PostgreSQL | Make sure the dependencies are installed
-  yum:
-    name: "{{ item }}"
-    state: present
-    update_cache: yes
-  with_items: ["python-psycopg2", "python-pycurl", "glibc-common"]
+      - name: PostgreSQL | Add PostgreSQL repository
+        yum:
+          name: "{{ postgresql_yum_repository_url }}"
+          state: present
 
-- name: PostgreSQL | Install PostgreSQL
-  yum:
-    name: "{{ item }}"
-    state: present
-  environment: "{{ postgresql_env }}"
-  with_items:
-    - "postgresql{{ postgresql_version_terse }}-server"
-    - "postgresql{{ postgresql_version_terse }}"
-    - "postgresql{{ postgresql_version_terse }}-contrib"
+      - name: PostgreSQL | Make sure the dependencies are installed
+        yum:
+         name: "{{ item }}"
+         state: present
+         update_cache: yes
+        with_items: ["python-psycopg2", "python-pycurl", "glibc-common"]
+
+      - name: PostgreSQL | Install PostgreSQL
+        yum:
+          name: "{{ item }}"
+          state: present
+        environment: "{{ postgresql_env }}"
+        with_items:
+          - "postgresql{{ postgresql_version_terse }}-server"
+          - "postgresql{{ postgresql_version_terse }}"
+          - "postgresql{{ postgresql_version_terse }}-contrib"
+
+      - name: PostgreSQL | PGTune
+        yum:
+          name: pgtune
+          state: present
+        environment: "{{ postgresql_env }}"
+        when: postgresql_pgtune
 
-- name: PostgreSQL | PGTune
-  yum:
-    name: pgtune
-    state: present
-  environment: "{{ postgresql_env }}"
-  when: postgresql_pgtune