[devel] pgbouncer

[Ivan Zakharyaschev]

On Fri, 14 Apr 2017, Ivan Zakharyaschev wrote:

> Свежесобранный пакет pgbouncer17 планируется переименовать в pgbouncer, чтобы 
> заменить старую версию 1.4.2.
>
> Есть у кого-нибудь замечания к pgbouncer17? (Опубликуется ночью.)

Уже есть такие замечания при попытке использования (больше тестировать 
нет смысла, смысл есть только исправлять пакет):

Не хватает /var/log/pgbouncer/

Нет .service-файла для systemd, поэтому systemctl идёт к 
/etc/init.d/pgbouncer, ждёт /var/run/pgbouncer.pid -- journalctl -r:

апр 14 23:06:44 db2.box.nct systemd[1]: pgbouncer.service: Failed with 
result 'protocol'.
апр 14 23:06:44 db2.box.nct systemd[1]: pgbouncer.service: Unit entered 
failed state.
апр 14 23:06:44 db2.box.nct systemd[1]: Failed to start Cluster 
Controlled pgbouncer.
апр 14 23:06:44 db2.box.nct systemd[1]: pgbouncer.service: Daemon never 
wrote its PID file. Failing.
апр 14 23:06:44 db2.box.nct systemd[1]: pgbouncer.service: PID file 
/var/run/pgbouncer.pid not readable (yet?) after start: No such file or 
directory
апр 14 23:06:44 db2.box.nct pgbouncer[24158]: Starting pgbouncer service: 
[ DONE ]
апр 14 23:06:44 db2.box.nct systemd[1]: Starting Cluster Controlled 
pgbouncer...

а сам процесс бы создал другой файл, если бы была директория:

# strace -Ff -e trace=file /etc/init.d/pgbouncer start 2>&1 | fgrep /var/
stat("/var/run/pgbouncer/pgbouncer.pid", 0x7ffea58174a0) = -1 ENOENT (No 
such file or directory)
...
[pid 24037] open("/var/run/pgbouncer/pgbouncer.pid", 
O_WRONLY|O_CREAT|O_EXCL, 0644) = -1 ENOENT (No such file or directory)

Не хватает /var/run/pgbouncer/

В пакете пакуется:

%define         pgb_runtimedir   /var/run/%name

т.е. /var/run/pgbouncer17/ (не то).

Да и паковать что-то в /var/run/ неправильно в соврменных системах 
(tmpfs), нужны ещё правила для tmpfiles.

Пока это успел заметить, но не успел исправить. Начал в 
git.alt:/people/imz/packages/pgbouncer17.git ветка alt. Там косметические 
коммиты, и я пока прекращаю вносить изменения.

-- 
Best regards,
Ivan