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

Added info #41

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
15 changes: 15 additions & 0 deletions case-study.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,15 @@
## Case-study

### О проекте
В данный момент мы занимаемся развитием цифрового банка. В проекте уже больше года, самому проекту чуть больше 2 лет. Проект состоит из большого монолита и нескольких микросервисов, для общения между которыми недавно стали использовать Kafka. В проекте используетс свежие Ruby/Rails, PostgreSQL.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

Используем только внутренние сервисы. Для метрик Yabeda + Grafana. Для мониторинга ElasticAPM, судя по которому с нагрузкой справляемся.
Встречаются в проекте иногда медленные sql запросы (помогает определить pghero), с которыми стараются сразу разобраться.
Я backend разработчик, вхожу в одну из продуктовых команд.

### Проблемы
Пока какаих то существенных проблем не было выявлено, но есть проблемы с тестами, который занимают уже больше 5 минут при использовании turbo_test.

### Решения
С проблема тестов помогло изучение test-prof с его инструментами, которые помогли найти проблемы в фабриках.
В рамках домашнего задания я оптимизировал свою часть тестов в проекте, что сократило общее время примерно на 20 секунд. Мои изменения одобрили, и согласовали дальнейшую работу.
Так же я обнаружил, что у нас в CI не эффективно используется turbo_test, мы не сохраняем лог его работы, и поэтому он не балансирует медленные тесты, этим я и собираюсь занятся в ближайшее время.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

На эту тему вот такая штука ещё есть: https://docs.knapsackpro.com/ruby/rspec/