Add Dockerfile & update compose.yaml #29
Conversation
AleksanderNekr
force-pushed
the
feat/docker-front
branch
from
4de3ec3 to
15a632a
Compare
|
У меня все поднялось, фронт заработал. |
| environment: | ||
| NODE_ENV: production | ||
| ports: | ||
| - 3001:3001 | ||
|
|
There was a problem hiding this comment.
нужно будеть добавить сеть, там в моем PR уже есть сеть для бека, нужно прокрутить сеть для фронт-бек
There was a problem hiding this comment.
Тут не понял, что имеешь в виду)
There was a problem hiding this comment.
Разобрался, ты, оказывается, в своей ветке компоуз для бд обновил, я прост не видел) Сделай пр, если готово
Насчет сети, можно добавить кнш, но у меня из без нее всегда работало, никогда не юзал и смысла особо не вижу)
Если нужно, могу добавить
There was a problem hiding this comment.
ну, всмысле, networking контейнеров (чтобы с фронта можно было подключиться к беку так: backend:3000/api), я сделаю, когда сольем все в кучу)
There was a problem hiding this comment.
Без этого сработает)
Вот пример
Там internal api взаимодействует c public api (gRPC вызовы друг другу), но никакой сети настраивать не нужно было
There was a problem hiding this comment.
Но давай сделаем, без проблем))
There was a problem hiding this comment.
@disk0Dancer, кстати, почитал доку, бридж по дефолту устанавливается, его можно явно не задавать, как я понимаю
@disk0Dancer, попробуй запустить докер, чтоб фронт тоже поднялся