[DPE-1755] Log rotation (#251)

* Initial implementation

* Log locations

* Fix log location

src/constants.py

@@ -21,9 +21,9 @@
 WORKLOAD_OS_USER = "postgres"
 METRICS_PORT = "9187"
 POSTGRES_LOG_FILES = [
-    "/var/log/pgbackrest",
+    "/var/log/pgbackrest/*",
     "/var/log/postgresql/patroni.log",
-    "/var/log/postgresql/postgresql.log",
+    "/var/log/postgresql/postgresql*.log",
 ]
 # List of system usernames needed for correct work of the charm/workload.
 SYSTEM_USERS = [BACKUP_USER, REPLICATION_USER, REWIND_USER, USER, MONITORING_USER]

templates/patroni.yml.j2

@@ -16,8 +16,25 @@ bootstrap:
         archive_command: /bin/true
         {%- endif %}
         archive_mode: on
-        log_filename: 'postgresql.log'
+        log_autovacuum_min_duration: 60000
+        log_checkpoints: 'on'
+        log_destination: 'stderr'
         log_directory: '/var/log/postgresql'
+        log_file_mode: '0600'
+        log_filename: 'postgresql-%w_%H%M.log'
+        log_hostname: 'off'
+        log_line_prefix: '%t [%p]: user=%u,db=%d,app=%a,client=%h,line=%l '
+        log_min_duration_sample: -1
+        log_recovery_conflict_waits: 'on'
+        log_replication_commands: 'on'
+        log_rotation_age: 1
+        log_rotation_size: 0
+        log_statement: ddl
+        log_statement_sample_rate: 1
+        log_statement_stats: 'off'
+        log_temp_files: 1
+        log_timezone: 'GMT'
+        log_truncate_on_rotation: 'on'
         logging_collector: 'on'
         wal_level: logical
   {%- if restoring_backup %}

tests/integration/test_tls.py

@@ -47,7 +47,7 @@ async def check_tls_rewind(ops_test: OpsTest) -> None:
             logs = await run_command_on_unit(
                 ops_test,
                 unit.name,
-                "grep rewind /var/log/postgresql/postgresql.log",
+                "grep rewind /var/log/postgresql/postgresql-*.log",
             )
         except Exception:
             continue