Size: a a a

2020 December 04

N

Nikki in aiogram [ru]
Просто указываешь порт конта и его имя
источник

VV

Vadim Vadim in aiogram [ru]
Евгений Петров
Просто уберите эту строчку с network_mode. Пусть они там в своей сети варятся
Видимо, дело не в этом было
Убрал эту строку, ошибки вообще не изменились
Залил код сюда, если можешь посмотри, пожалуйста  https://yadi.sk/d/9xiT2r6iHBdTjQ?w=1
источник

ЕП

Евгений Петров... in aiogram [ru]
это прекрасно
источник

VV

Vadim Vadim in aiogram [ru]
😐Там сложная ситуация
С багом не хочу коммитить, а проект уже есть
источник

ЕП

Евгений Петров... in aiogram [ru]
Можно же в отдельную ветку запушить. Например, "testing"
источник

VV

Vadim Vadim in aiogram [ru]
источник

ЕП

Евгений Петров... in aiogram [ru]
Пока не забыл: https://github.com/Windmen05/gender_age_bot/blob/docker_bug/Dockerfile#L6
5 и 6 строки лучше поменять местами (как минимум), чтобы не переустанавливать библиотеки каждый раз при изменении в коде
источник

VV

Vadim Vadim in aiogram [ru]
Поменял, большое спасибо
А то что я docker build каждый раз после изменения делаю, это же нормально?
источник

ЕП

Евгений Петров... in aiogram [ru]
По https://github.com/Windmen05/gender_age_bot/blob/docker_bug/docker-compose.yml

1) Давайте для начала уберём container_name вообще. Тогда в качестве хоста надо будет использовать "db"
2) Давайте уберём прокидывание порта 5432, вряд ли надо лезть в БД с хоста. Бот и БД друг друга увидят и так
3) Да, каждый раз делать docker build нормально, чтобы в образ попали изменённые версии py-файлов
источник

ЕП

Евгений Петров... in aiogram [ru]
Ну и я бы ещё поставил версию compose повыше (3.7, например). Разницы там немного, в подробности не вдавался, сам использую 3.8 (это НЕ версия Python, если что)
источник

ЕП

Евгений Петров... in aiogram [ru]
"env_file" в данном случае тоже лишний, т.к. compose автоматически подцепит .env при наличии
источник

VV

Vadim Vadim in aiogram [ru]
Евгений Петров
Пока не забыл: https://github.com/Windmen05/gender_age_bot/blob/docker_bug/Dockerfile#L6
5 и 6 строки лучше поменять местами (как минимум), чтобы не переустанавливать библиотеки каждый раз при изменении в коде
ERROR: Could not open requirements file: [Errno 2] No such file or directory: 'requirements.txt'
После изменений
источник

ЕП

Евгений Петров... in aiogram [ru]
Vadim Vadim
ERROR: Could not open requirements file: [Errno 2] No such file or directory: 'requirements.txt'
После изменений
А, лул.
COPY requirements.txt /src/ надо отдельной командой сделать, конечно же. Иначе pip install будет выполняться ДО того, как все файлы будут скопированы. Поэтому сам requirements.txt надо положить раньше, чем pip install, а всё остальное можно позже

Или вообще multi-stage build.
Например, такой (не надо его копипастить вслепую): https://gitlab.com/groosha/telegram-stats-demo/-/blob/master/Dockerfile (чтобы в целевом контейнере ничего не собирать лишний раз)
источник

VV

Vadim Vadim in aiogram [ru]
Действительно, собирается в разы быстрее, спасибо
После всех изменений он все еще пишет [Errno -2] Name or service not known
источник

ЕП

Евгений Петров... in aiogram [ru]
Изменения закоммичены?
источник

Ü

Ü in aiogram [ru]
у кого нибудь cmd окончательно не разваливалась?
источник

VV

Vadim Vadim in aiogram [ru]
Да
источник

Ü

Ü in aiogram [ru]
вообще ни на какую нкоманду не реагирует
источник

Ü

Ü in aiogram [ru]
сотни путей уже перебрал для patha
источник

Ü

Ü in aiogram [ru]
благодаря этому опыту я понимаю что убунту лучше несколько раз
источник