Bug 34367

Summary: Лишние файлы
Product: Sisyphus Reporter: Стас <stas.grumbler>
Component: postgresql-commonAssignee: Alexei Takaseev <taf>
Status: CLOSED FIXED QA Contact: qa-sisyphus
Severity: normal    
Priority: P3 CC: mike, taf
Version: unstable   
Hardware: all   
OS: Linux   
URL: http://git.altlinux.org/tasks/197156/
Attachments:
Description Flags
Убираем control и /etc/sysconfig/postgresql none

Description Стас 2017-12-20 09:58:55 MSK
1. Пакет ставит /etc/sysconfig/postgresql с единственной строкой
PG_CHROOT_DIR=/var/lib/pgsql-root
Причём стартовый скрипт не обращается к /etc/sysconfig/postgresql и переменная PG_CHROOT_DIR не используется стартовым скриптом
По обсуждению в [comm]:
"PostgreSQL сам по себе перестал жить в chroot, ЕМНИП, начиная с 9.1. PG для 1С
в chroot не помещался от рождения." - https://lists.altlinux.org/pipermail/community/2017-December/687052.html
Предлагаю патч , убирающий остатки chrooted postgresql

2. Пакет ставит /etc/monitrc.d/postgresql
Хорошо бы убрать его в отдельный пакет, например, postgresql-monit. Тогда postgresql не будет тащить за собой monit.

3. Может быть создание пользователя имеет смысл перенести в пакеты posgres-server* ?
Comment 1 Стас 2017-12-20 13:29:42 MSK
Created attachment 7325 [details]
Убираем control и /etc/sysconfig/postgresql
Comment 2 Стас 2017-12-20 13:32:27 MSK
Комментарий по конфигу monit: "И ещё я бы оторвал дописывание в конфиг монита расположение пид-файла - ну ломает же настройку." - https://lists.altlinux.org/pipermail/community/2017-December/687067.html
Comment 3 Michael Shigorin 2017-12-20 17:42:20 MSK
(В ответ на комментарий №0)
> 2. Пакет ставит /etc/monitrc.d/postgresql
> Хорошо бы убрать его в отдельный пакет, например, postgresql-monit.
Зачем?

> Тогда postgresql не будет тащить за собой monit.
Он и не тащит, так задумано.
Comment 4 Стас 2017-12-20 18:12:44 MSK
(In reply to comment #3)
> (В ответ на комментарий №0)
> > 2. Пакет ставит /etc/monitrc.d/postgresql
> > Хорошо бы убрать его в отдельный пакет, например, postgresql-monit.
> Зачем?
> 
> > Тогда postgresql не будет тащить за собой monit.
> Он и не тащит, так задумано.

Тут я ошибся, зависимость от monit-base
Comment 5 Michael Shigorin 2017-12-20 18:28:13 MSK
(В ответ на комментарий №4)
> > > Тогда postgresql не будет тащить за собой monit.
> > Он и не тащит, так задумано.
> Тут я ошибся, зависимость от monit-base
Она действительно так страшна? :)

$ rpm -qRl --qf='%{SIZE}\n' monit-base-5.25.1-alt1
rpmlib(PayloadIsLzma)  
0
/etc/monitrc.d
$ _