Add docker compose configs, dockerfile, update readme. #10
Conversation
| retries: 5 | ||
| volumes: | ||
| - ./postgres-data:/var/lib/postgresql/data | ||
| - ./postgres-firstrun/init-user-db.sh:/docker-entrypoint-initdb.d/init-user-db.sh:ro |
There was a problem hiding this comment.
В целом, я бы вообще отказался от каких-то скриптов инициализации. М.б. проще дать юзеру самому создать всё необходимое? Роли, юзеры... наверное, это их дело
There was a problem hiding this comment.
Этот момент я не правил. Так было в docker-compose.example.yml
И скрипт был создан, до моих правок.
В этом скрипте идет создание БД и юзера для БД при первом включении.
Можно действительно обойтись без этого скрипта, просто описав в readme, мол создайте БД, создайте юзера, и дальше вот в этом и этом месте пропишите эти данные.
Мне, наоборот, нравится со скриптом - это ускоряет разворачивание проекта.
|
|
||
| ⚠️ **Warning**: project is still under development, use with caution. | ||
| [Issues](https://github.com/MasterGroosha/telegram-feedback-bot-topics/issues) are greatly appreciated! | ||
| ⚠️ **Warning**: project is still under development, use with caution. | ||
| [Issues](https://github.com/MasterGroosha/telegram-feedback-bot-topics/issues) are | ||
| greatly appreciated! | ||
|
|
There was a problem hiding this comment.
Что-то я не вижу здесь различий
There was a problem hiding this comment.
Похоже, что просто из-за переноса строки.... У меня IDE автоматически делает переносы, если длина строки превышает 88 символов (black).
Remove old docker-compose example file.
No description provided.