Use the the original PostgreSQL configuration file

defaults/main.yml

@@ -31,6 +31,7 @@ postgresql_server_auth: []
 # Recursively reset the owner/group of the postgres datadir?
 postgresql_server_chown_datadir: false
 
+postgres_config_file_contents: ''
 ######################################################################
 # Internal role variables, do not modify
 ######################################################################
@@ -58,7 +59,7 @@ postgresql_dist_redhat:
 postgresql_dist_debian:
   bindir: /usr/lib/postgresql/{{ postgresql_version }}/bin
   confdir: /etc/postgresql/{{ postgresql_version }}/main
-  conf_postgresql_src: postgresql-conf-10-ubuntu.j2
+  conf_postgresql_src: postgresql-conf.j2
   datadir: /var/lib/postgresql/{{ postgresql_version }}/main
   basename: postgresql-{{ postgresql_version }}
   service: postgresql

molecule/resources/tests/test_default.py

@@ -48,14 +48,23 @@ def test_server_listen(host):
     assert listen_addresses == "listen_addresses = localhost"
 
 
+def test_backup_config_exist(host):
+    version = get_version(host)
+    if host.system_info.distribution == 'rocky':
+        config_backup = '/var/lib/pgsql/{version}/data/postgresql.conf.backup'
+    else:
+        config_backup = '/etc/postgresql/{version}/main/postgresql.conf.backup'
+    with host.sudo():
+        backup_file = config_backup.format(version=version)
+        assert host.file(backup_file).is_file
+
+
 def test_psql_version(host):
     ver = get_version(host)
     out = host.check_output('psql --version')
     assert out.startswith('psql (PostgreSQL) {}.'.format(ver))
 
 
-# Create
-
 def createdb(host, db, should_pass, password, name):
     try:
         host.check_output(

molecule/resources/tests/test_extra_options.py

@@ -23,9 +23,9 @@ def test_server_log_file_name(host):
     # Check previous day too in case this is run at midnight
     version = get_version(host)
     if host.system_info.distribution == 'rocky':
-        logdir = '/var/lib/pgsql/{version}/data/pg_log'
+        logdir = '/var/lib/pgsql/{version}/data/log'
     else:
-        logdir = '/var/lib/postgresql/{version}/main/pg_log'
+        logdir = '/var/lib/postgresql/{version}/main/log'
     date1 = datetime.today()
     date0 = date1 - timedelta(days=1)
     logdir = logdir.format(version=version)

tasks/initialise.yml

@@ -22,27 +22,78 @@
         PGSETUP_INITDB_OPTIONS: >-
           --encoding=UTF8 --locale=en_US.UTF-8 --auth-host=md5
 
-    - name: postgres | postgresql config file
-      template:
+    - name: postgres | Check for presence of "Modified by ome postgresql ansible role" in file
+      ansible.builtin.lineinfile:
+        path: "{{ postgresql_dist_confdir }}/postgresql.conf"
+        line: "#Modified by ome postgresql ansible role"
+        state: absent
+      check_mode: yes
+      changed_when: false
+      register: config_file_changed
+
+    - name: postgres | Check that the postgresql.conf.backup file exists
+      ansible.builtin.stat:
+        path: "{{ postgresql_dist_confdir }}/postgresql.conf.backup"
+      register: backup_result
+
+
+    # read the default postgresql configuration file
+    - name: postgres | get the postgres conf file contents
+      become_user: "{{ postgresql_become_user }}"
+      ansible.builtin.slurp:
+        src: "{{ postgresql_dist_confdir }}/postgresql.conf"
+      register: postgres_config_file_contents_o
+      when:
+        - not backup_result.stat.exists
+
+    # read the default postgresql configuration file
+    - name: postgres | get the postgres conf file contents from backup
+      become_user: "{{ postgresql_become_user }}"
+      ansible.builtin.slurp:
+        src: "{{ postgresql_dist_confdir }}/postgresql.conf.backup"
+      register: postgres_config_file_contents_b
+      when: backup_result.stat.exists
+
+    - set_fact: postgres_config_file_contents={{ postgres_config_file_contents_o }}
+      when:
+        - not backup_result.stat.exists
+
+    - set_fact: postgres_config_file_contents={{ postgres_config_file_contents_b }}
+      when:
+        - backup_result.stat.exists
+
+    - name: postgres | Copy a postgresql.conf to postgresql.conf.backup
+      become_user: "{{ postgresql_become_user }}"
+      ansible.builtin.copy:
+        src: "{{ postgresql_dist_confdir }}/postgresql.conf"
+        dest: "{{ postgresql_dist_confdir }}/postgresql.conf.backup"
+        remote_src: yes
+      when:
+        - not backup_result.stat.exists
+        - config_file_changed.found | default(0) == 0
+
+    - name: postgres | copy postgresql config file
+      become_user: "{{ postgresql_become_user }}"
+      ansible.builtin.template:
         dest: >-
           {{ postgresql_dist_confdir }}/postgresql.conf
         src: "{{ postgresql_dist_conf_postgresql_src }}"
         mode: 0644
+        owner: "{{ postgresql_become_user }}"
       notify:
         - restart postgresql
-
-      become_user: "{{ postgresql_become_user }}"
+      when:
+        - config_file_changed.found | default(0) == 0 or backup_result.stat.exists
 
     - name: postgres | configure client authorisation
+      become_user: "{{ postgresql_become_user }}"
       template:
         dest: "{{ postgresql_dist_confdir }}/pg_hba.conf"
         src: pg_hba-conf.j2
         mode: 0640
       notify:
         - restart postgresql
 
-      become_user: "{{ postgresql_become_user }}"
-
     - name: postgres | start service
       service:
         enabled: true