mirror of
https://github.com/werwolfby/monitorrent.git
synced 2026-10-14 21:39:03 +00:00
[GH-ISSUE #361] 1.2.0 Docker doesn't start #290
Labels
No labels
bug
docs
duplicate
enhancement
Epic
feature
feature
future
help wanted
in progress
need more info
plugin
proxy
pull-request
python 2
python 3
question
wontfix
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
werwolfby/monitorrent#290
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @Egorkat on GitHub (Mar 30, 2022).
Original GitHub issue: https://github.com/werwolfby/monitorrent/issues/361
Log contains this repeating:
Traceback (most recent call last):
File "/var/www/monitorrent/server.py", line 13, in
import structlog
ModuleNotFoundError: No module named 'structlog'
@werwolfby commented on GitHub (Mar 30, 2022):
Странно
structlogдолжен быть только вdevelopветке. А я собирал все сmaster, перепроверю.@werwolfby commented on GitHub (Mar 30, 2022):
@Egorkat а командой для запуска поделитесь?
@Egorkat commented on GitHub (Mar 30, 2022):
'python' 'server.py'
@werwolfby commented on GitHub (Mar 30, 2022):
Для всего докера
@werwolfby commented on GitHub (Mar 30, 2022):
Типа:
docker run -v … werwolfby/monitorrent:1.2.0@Egorkat commented on GitHub (Mar 30, 2022):
я его через portainer запускаю, предыдущая версия работала с текущими настройками. Попробую вытащить лог запуска
@werwolfby commented on GitHub (Mar 30, 2022):
Ну скриншот портейнера можно, с запуском приложения. Мне не нужно команда внутри контейнера мне нужно понять какие волумы замаунчены в контейнер.
Я проверил контейнер в файле
server.pyнет импортаstructlog. Такое есть только в девелоп ветке. Так что вижу только вариант когда сырцы почему-то перемаунчены…Хотя… есть шанс что
monitorrent-1.2.0.zipпочему-то с девелопера (и это я могу объяснить и пофиксить), сейчас проверю@werwolfby commented on GitHub (Mar 30, 2022):
Да, так и есть. Но это значит что вы что-то сами руками собираете, а не используете готовый образ.
@Egorkat commented on GitHub (Mar 30, 2022):
куда мне своими руками, портэйнер тянет свежий образ с докерхаба
@werwolfby commented on GitHub (Mar 30, 2022):
Блин ну я сейчас перепроверю еще раз. Но скриншот того как контейнер запускается сильно помог бы.
@Egorkat commented on GitHub (Mar 30, 2022):
@werwolfby commented on GitHub (Mar 30, 2022):
Вон в самом низу видна проблема: вы моунтите всю папку
Monitorrentв/var/www/monitorrent. Тем самым перетирая все сырцы что уже есть в контейнере. Вам либо в папку зобрать все сырцы из мастер ветки, либо маунтить только файлmonitorrent.dbвместо всей папке как и написано в ридми.Попробуйте для начала поменять на файл только, уверен что все сразу заработает как надо.
@werwolfby commented on GitHub (Mar 30, 2022):
Лидо если
Monitorrentэто докер volume. То можно извратнуться:Сделать моунт вместо
/var/www/monitorrentв/var/www/old-monitorrent(old-monitorrentпросто имя для примера.И выставить новую ENV переменную
MONITORRENT_DB_PATH=/var/www/old-monitorrent/monitorrent.db. Тогда старый волюм будет задействован но мы из него только базу будем брать, а новый сырцы будут браться из контейнера.@werwolfby commented on GitHub (Mar 30, 2022):
У вас раньше конечно все работало, но правильно все было только если вместе с контейнером вы сырцы в волюме тоже апдейтали из правильного тега, который совпадает с тегом контейнера (но вы юзаете
latestкстати)@Egorkat commented on GitHub (Mar 30, 2022):
да, это volume. С путем просто к базе не стартует. Попробую "извратнуться" :-)
@werwolfby commented on GitHub (Mar 30, 2022):
Да волюм - это всегда «папка» так что так просто и не стартанет.
@Egorkat commented on GitHub (Mar 30, 2022):
Завелось! в лог поругалоссь только:
/usr/local/lib/python3.9/site-packages/alembic/util/messaging.py:73: UserWarning: 'connection' argument to configure() is expected to be a sqlalchemy.engine.Connection instance, got <sqlalchemy.orm.session.ContextSession object at 0x7fad7e49dd90>
@werwolfby commented on GitHub (Mar 30, 2022):
Это стандартный варнинг можно игнорить :)
Закрывайте ишью, а то 100 - было хорошим числом :)
@werwolfby commented on GitHub (Mar 30, 2022):
Ну и да, моунтить чисто файл не очень хорошая идея, надо пофиксить. И проапдейтить ридми, хотя это может сломать совместимость.
Надо было изначально правильно сделать, но мне казалось очень привлекательной идеей моунтить чисто файл с базой и не заморачиваться с папками.