zalando / spilo

Highly available elephant herd: HA PostgreSQL cluster using Docker
Apache License 2.0
1.56k stars 387 forks source link

Use relevant params in pgqd ini file #842

Closed hughcapet closed 1 year ago

hughcapet commented 1 year ago

Seems, config file didn't set anything for a long time (if ever did).

mblsf commented 1 year ago

I think the new parameters logfile and pidfile are also mandatory

hughcapet commented 1 year ago

@mblsf as far as I see, pidfile is only required for daemonizing. But I think logfile is worth using together with pgqd -q in order to avoid mixing up pgqd and Patroni output in container logs, thanks for pointing to it.

sdudoladov commented 1 year ago

:+1:

hughcapet commented 1 year ago

👍