Conversation
taaylor
left a comment
taaylor
left a comment
There was a problem hiding this comment.
Спасибо, очень круто все покрыл. Красота!
|
|
||
|
|
||
| class EmbeddingApi(BaseModel): | ||
| host: str = "embedding-service" |
There was a problem hiding this comment.
Там из .env просто не читает почему то, и я так бахнул) Попытаюсь пофиксить
|
|
||
| response = sock.recv(1024).decode() | ||
| code = response.split(" ")[1] | ||
| if code == "200": |
There was a problem hiding this comment.
[придирка] захардкоржено (статус код), правда я не помню как в других тестах выглядит этот момент. Если там также то можно оставить
There was a problem hiding this comment.
Также, я просто копировал, и немного изменил в другом месте. Вроде это из-за того что в socket статус код отдаётся как str в не int
|
|
||
| FROM python:${PYTHON_VERSION}-slim | ||
|
|
||
| # ! -=-=-=-=- моя секция -=-=-=-=- |
There was a problem hiding this comment.
Убрать коммент
There was a problem hiding this comment.
Мне кажется задачу этого файла в принципе выполняет хелсчек.
There was a problem hiding this comment.
Да вроде даже нет, потому что он сразу падал почему то, т.е. не работал как хелсчек. Думаю можно убрать его
|
|
||
|
|
||
| def backoff( | ||
| exception: tuple[Type[Exception], ...], |
There was a problem hiding this comment.
Использовать backoff из библиотеки
3 participants
Прогон тестов добавил в main.yml для CI пайплайна, надеюсь всё нормально (переживаю)