From ca25b5ce7c5f7b0a798535ec234624d2c4d2d135 Mon Sep 17 00:00:00 2001 From: Philipp Eberle Date: Wed, 16 Jul 2025 16:34:28 +0200 Subject: [PATCH 1/2] Fine tune logging --- api/v1/postgres_types.go | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/api/v1/postgres_types.go b/api/v1/postgres_types.go index a479aafc..1d04380c 100644 --- a/api/v1/postgres_types.go +++ b/api/v1/postgres_types.go @@ -90,6 +90,7 @@ const ( defaultPostgresParamValueLogMinErrorStatement = "WARNING" defaultPostgresParamValueLogErrorVerbosity = "VERBOSE" defaultPostgresParamValueLogLinePrefix = "%m [%p]: [%l-1] db=%d,user=%u,app=%a,client=%h " + defaultPostgresParamValueLogStatement = "none" // PostgresAutoAssignedIPNamePrefix a prefix to add to the generated random name PostgresAutoAssignedIPNamePrefix = "pgaas-autoassign-" @@ -962,7 +963,7 @@ func enableAuditLogs(parameters map[string]string) { // default values: bg_mon,pg_stat_statements,pgextwlist,pg_auth_mon,set_user,timescaledb,pg_cron,pg_stat_kcache parameters["shared_preload_libraries"] = "pg_stat_statements,pgextwlist,pg_auth_mon,set_user,timescaledb,pg_cron,pg_stat_kcache,pgaudit" parameters["pgaudit.log_catalog"] = "off" - parameters["pgaudit.log"] = "ddl" + parameters["pgaudit.log"] = "ddl,role" parameters["pgaudit.log_relation"] = "on" parameters["pgaudit.log_parameter"] = "on" } @@ -980,6 +981,7 @@ func setDefaultPostgresParams(parameters map[string]string, version string) { parameters["ssl_prefer_server_ciphers"] = defaultPostgresParamValueSSLPreferServerCiphers parameters["tcp_keepalives_idle"] = defaultPostgresParamValueTCPKeepAlivesIdle parameters["tcp_keepalives_interval"] = defaultPostgresParamValueTCPKeepAlivesInterval + parameters["log_statement"] = defaultPostgresParamValueLogStatement // set version specific parameters v, err := strconv.Atoi(version) From 9e4f69404332abbefecea51d4f00a65211998d9c Mon Sep 17 00:00:00 2001 From: Philipp Eberle Date: Thu, 17 Jul 2025 15:19:53 +0200 Subject: [PATCH 2/2] Move log_statement after discussion --- api/v1/postgres_types.go | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/api/v1/postgres_types.go b/api/v1/postgres_types.go index 1d04380c..e65aa15d 100644 --- a/api/v1/postgres_types.go +++ b/api/v1/postgres_types.go @@ -90,7 +90,6 @@ const ( defaultPostgresParamValueLogMinErrorStatement = "WARNING" defaultPostgresParamValueLogErrorVerbosity = "VERBOSE" defaultPostgresParamValueLogLinePrefix = "%m [%p]: [%l-1] db=%d,user=%u,app=%a,client=%h " - defaultPostgresParamValueLogStatement = "none" // PostgresAutoAssignedIPNamePrefix a prefix to add to the generated random name PostgresAutoAssignedIPNamePrefix = "pgaas-autoassign-" @@ -966,6 +965,7 @@ func enableAuditLogs(parameters map[string]string) { parameters["pgaudit.log"] = "ddl,role" parameters["pgaudit.log_relation"] = "on" parameters["pgaudit.log_parameter"] = "on" + parameters["log_statement"] = "none" } // setDefaultPostgresParams configures default keepalive values @@ -981,7 +981,6 @@ func setDefaultPostgresParams(parameters map[string]string, version string) { parameters["ssl_prefer_server_ciphers"] = defaultPostgresParamValueSSLPreferServerCiphers parameters["tcp_keepalives_idle"] = defaultPostgresParamValueTCPKeepAlivesIdle parameters["tcp_keepalives_interval"] = defaultPostgresParamValueTCPKeepAlivesInterval - parameters["log_statement"] = defaultPostgresParamValueLogStatement // set version specific parameters v, err := strconv.Atoi(version)