Skip to content

Conversation

v-loboda
Copy link
Collaborator

@v-loboda v-loboda commented Sep 8, 2025

Pull Request description

Changelog

Issues

Breaking changes

  • Настройка kafka.eventsTopic.readerGroupId теперь обязательна для tasks-worker
  • keys полностью удалены из values, теперь у нас нет прямой зависимости от "старого" сервиса ключей

Check-list. Чек-лист код-ревью

  • Запрос на слияние в develop.
  • Есть описание к PR.
  • Указаны блокирующие изменения. Breaking-Changes
  • Соответствие кода принятому стилю
    • Описание настроек.
    • Именование настроек.
    • Дефолтные значения.
    • Стиль кода.
  • Работоспособность. Разворачивается на своем окружении из ветки PR.
    • Тест API через тесты helmfile-хуков или коллекций Postman.
  • Не осталось мусора от удаления каких-то параметров. Ищется поиском по проекту из ветки PR.
  • Отработка линтера на чарт из ветки PR. Пример: helm lint charts/search-api

@v-loboda v-loboda requested review from a team as code owners September 8, 2025 04:34
@dbelyaev-nsk
Copy link
Contributor

задача на проверку https://jira.2gis.ru/browse/ONPREM-2440

@r9odt
Copy link
Collaborator

r9odt commented Oct 6, 2025

Нужно актуализировать PR с решением конфликтов

@v-loboda
Copy link
Collaborator Author

v-loboda commented Oct 7, 2025

Нужно актуализировать PR с решением конфликтов

Ребят, я напоминаю, что мы используем ровно тот же чарт, что и на бою, т.е. конфликты исключены, при условии, ЧТО ВЫ НЕ ВНОСИТЕ ИЗЕНЕНИЙ в чарты. А если вносите, то сообщаете нам об этом. Не далайте так, пожалуйста. Теперь мне сначала придётся внести те же самые изменения в наш чарт, и потом перенести всё сюда. Придётся подождать, когда я смогу добраться до этого,

@v-loboda
Copy link
Collaborator Author

v-loboda commented Oct 7, 2025

ветку актуализировал

@dbelyaev-nsk dbelyaev-nsk merged commit d72e6cb into develop Oct 9, 2025
2 checks passed
@dbelyaev-nsk dbelyaev-nsk deleted the PRO-9224-ProApi branch October 9, 2025 08:07
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.

3 participants