Change minimum log file size to 1k and remove create option (implied by sharedscripts). Use pkill command for postrotate script.

Author: dsessler7

internal/collector/logrotate.conf

@@ -1,25 +1,12 @@
 %s {
-
       rotate %s
-
       missingok
-
       sharedscripts
-
       notifempty
-
       nocompress
-
-      size 10m
-
+      size 1k
       %s
-
-      create 0640 postgres postgres
-
       postrotate
-
            %s
-
       endscript
-
 }

internal/collector/pgbouncer.go

@@ -193,8 +193,8 @@ func AddPgBouncerLogrotateConfig(ctx context.Context, outInstanceConfigMap *core
 
 	// FIXME: get retentionPeriod from instrumentationSpec
 	pgbouncerLogPath := naming.PGBouncerLogPath + "/pgbouncer.log"
-	retentionPeriod := "1d"
-	postrotateScript := "/bin/kill -HUP `cat /var/pgbouncer-postgres/pgbouncer.pid 2> /dev/null` 2>/dev/null ||true"
+	retentionPeriod := "1h"
+	postrotateScript := "pkill -HUP --exact pgbouncer"
 
 	logrotateConfig, err := generateLogrotateConfig(pgbouncerLogPath, retentionPeriod, postrotateScript)
 	if err != nil {