Re: Reloading pgbouncer and systemd - Mailing list pgsql-pkg-yum

From Jehan-Guillaume de Rorthais
Subject Re: Reloading pgbouncer and systemd
Date
Msg-id 20160330141641.61f28755@firost
Whole thread Raw
In response to Reloading pgbouncer and systemd  (Jehan-Guillaume de Rorthais <jgdr@dalibo.com>)
List pgsql-pkg-yum
Le Mon, 22 Feb 2016 17:38:45 +0100,
Jehan-Guillaume de Rorthais <jgdr@dalibo.com> a écrit :

> Hello,
>
> While working with pgbouncer under CentOS7, I noticed an incompatibility
> between pgbouncer and systemd when reloading.
>
> The current reload command is:
>
>   ExecReload=/usr/bin/pgbouncer -R -q ${BOUNCERCONF}
>
> But this command create a new pgbouncer process...and keeps it in foreground.
> As the service type is set as "forking", in this situation systemctl just sits
> and wait for a return code (maybe there's a timeout, but I did not have the
> patience to wait for it).
>
> Either add a '-d' to this command, or send a HUP signal to the process.
>
> I prefer the second option as it doesn't create a new process that takes over
> the sockets, but just reloads the conf file as expected. Much lighter.
>
> So, imho, the pgbouncer.service file should have:
>
>   ExecReload=/bin/kill -HUP $MAINPID

Any feedback on this ?


pgsql-pkg-yum by date:

Previous
From: Jeff Frost
Date:
Subject: Re: New releases are due on Thursday
Next
From: Devrim Gündüz
Date:
Subject: Re: New releases are due on Thursday