Skip to content
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

15 q1 not found #18

Merged
merged 11 commits into from
Oct 9, 2023
Merged

15 q1 not found #18

merged 11 commits into from
Oct 9, 2023

Conversation

audetv
Copy link
Contributor

@audetv audetv commented Oct 8, 2023

Добавлено условие проверки ошибки, отсутствия очереди на сервере сообщений, в случае отсутствия производится выход из системы для перезапуска контейнера.
Удален параметр отвечающий за кол-во поток рестарта сервиса max_attempts, таким образом докер всегда будет пытаться перезапустить сломанный контейнер (в случае ошибки).

Есть идея добавить на сервис svodd cli-php функцию, которая при старте будет создавать очередь. Сейчас он это делает только, когда поступил новый комментарий. Поэтом возникают случаи, когда запуск всех систем состоялся и успешен, но очередь еще не создана, т.к. после рестарта не было еще новых сообщений. Надо принудительно при рестарте системы проверять и создавать очередь. тогда шанс возникновения такой ошибки будет сведен к минимуму

@audetv
Copy link
Contributor Author

audetv commented Oct 8, 2023

Ошибка при сборке приложения:

Error: exitAfterDefer: os.Exit will exit, and `defer func(subs *pubsub.Subscription, ctx context.Context){...}(...)` will not run (gocritic)
Error: issues found

В общем то все верно. Надо подумать как сделать это по другому. Возможно стоит начать с реализации на сервисе svodd cli-php

@audetv
Copy link
Contributor Author

audetv commented Oct 9, 2023

Код переработан.
Ошибки из пакета msgreceiver передаются обратно в main и там обрабатываются: логируются, передаются в sentry, и осуществляется выход из приложения с ненулевым кодом, для того чтобы инициировать перезапуск докер контейнера.

Параметры перезапуска докер контейнера обновлены, теперь система без ограничений будет пытаться запустить сервис tg-svodd-bot

@audetv audetv merged commit fb9eaa0 into main Oct 9, 2023
2 checks passed
@audetv audetv deleted the 15-q1-not-found branch October 9, 2023 14:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Проблема после перезагрузки docker контейнеров
1 participant