-
Notifications
You must be signed in to change notification settings - Fork 5
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Feachure/postgres local stage #70
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ну и собственно не хватает самого конфига джанги) Где идет подключение к postgres. Ну с этим понятно, можно чуть подождать, когда следующие изменения подтянуться. Но разобраться не помешает сейчас
RUN poetry config virtualenvs.in-project true | ||
RUN poetry install --with dev | ||
|
||
COPY ./entrypoint.sh /entrypoint.sh |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
entrypoint.sh давай удалим отсюда и сам файл тоже. Он нам ник чему
COPY ./pyproject.toml ./ | ||
|
||
RUN python -m pip install --upgrade pip | ||
RUN python -m pip install --no-cache-dir poetry==1.7.1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ты в итоге тут ставишь poetry. Хоть это и локальная версия, но нам в принципе для контейнера он не нужен. обойтись можно стандартным pip. У нас для это в pre-commit делается экспорт различных пакетов зависимостей
requirements/develop.txt
посмотри файл .pre-commit-config.yaml
Т.е. нужно скопировать этот txt в контейнер и установить по нему
|
||
volumes: | ||
postgres_data: | ||
db_data: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
лучше postgres_db_data. Удобнее для масштабирования проекта. Вдруг еще какая то БД добавиться, чтобы различать их
@blizard-sis Зачем ты закрыл пул реквест? Его просто нужно было обновлять новыми коммитами в ветку свою же. |
Description
создан и оттестирован docker-compose file для разработки dev. ПР необходим для дальнейшей имплементации в Makefile
Type of change
Пожалуйста, удалите варианты, которые не относятся к ПР-у.
How Has This Been Tested?
запуск контейнера локально проверяется через команду в корне проекта
для запуска необходимо перенести .env.example в директорию
/infra/dev/
и переименовать в.env
Checklist: