mamonsu icon indicating copy to clipboard operation
mamonsu copied to clipboard

Невозможность запуска mamonsu от пользователя, отличного от mamonsu

Open xinferum opened this issue 3 years ago • 3 comments

Добрый день.

Версия mamonsu: 3.5.2

Мы используем у себя свой кастомный плагин для мониторинга на серверах бекапов pg_probackup. После обновления до версии 3.5.2 плагин перестал корректно работать, не отдает данные об имеющихся бекапов. В логе обнаружил ошибки доступа к файлам:

[ERROR] 2022-09-14 03:41:03,948 - PGPROBACKUP   -   The command: ['/usr/bin/pg_probackup-14', 'show', '-B', '/mnt/pgbak', '--format=json'] return code 1. Error: b'ERROR: Cannot open directory "/mnt/pgbak/backups": Permission denied\n'

Дело в том, что файлы в бекапе доступны только владельцу - postgres, и поменять это мы не имеем возможности (стоит еще учитывать что сам pg_probackup те же wal файлы всегда пишет из под postgres с правами 600) и ранее у нас mamonsu запускался от пользователя postgres и все работало. Сейчас же, мы переделали в сервисе запуск mamonsu от пользователя mamonsu, т.к. в новой версии введена проверка на владельца и права файла agent.conf: https://github.com/postgrespro/mamonsu/blob/9bd4952b674a93d64459d8332fb350ca07a14495/mamonsu/lib/config.py#L87-L94

И получилось что у нас плагин потерял возможность ходить и мониторить бекапы. Запустить же mamonsu как раньше, от пользователя postgres, не позволяет выше приведенная проверка, т.к. запуск от пользователя, отличного от mamonsu, требует поменять либо владельца agent.conf, либо права изменить, но это все приводит к завершению mamonsu при старте из-за выше приведенной проверки.

Можно ли на уровне кода установить, чтобы хотя бы были доступны вариации по правам и владельцу файла agent.conf? Например чтобы он мог иметь права не только 600, но и 644. И нам так же требуется чтобы можно было бы чтобы владельцем файла мог бы быть и пользователь postgres (помимо mamonsu), т.к. в контейнерах Docker agent.conf именно с владельцем postgres разворачивается.

В текущей ситуации, нам придется перед сборкой из исходников самим внедряться в код и изменять там этот момент.

xinferum avatar Sep 14 '22 09:09 xinferum

Добрый день. Столкнулись с точно такой же проблемой. Хотелось бы конечно получить внятный комментарий от разработчиков.

levinsv avatar Nov 29 '22 09:11 levinsv

Мы в итоге (т.к. собираем mamonsu из исходников для своих нужд), вносим автоматически правки в эту часть кода, чтобы работало так как нам надо. Разрешили нужные нам права на конфигурационный файл (600, 644) и пользователей postgres и root добавили (помимо mamonsu) в условиях if. Пока так пришлось сделать.

xinferum avatar Nov 29 '22 09:11 xinferum

Добрый день! @xinferum @levinsv в последнем вышедшем патче эту проверку убрали. В планах сделать проверку более гибкой и не привязываться к конкретному юзеру mamonsu, а определять запускающего пользователя.

cuprumtan avatar Mar 16 '23 10:03 cuprumtan