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.

Sign up for GitHub

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

Jump to bottom

[PRO-6515] Обновление чарта pro-api #548

Merged
merged 9 commits into from
Dec 26, 2024
Merged

[PRO-6515] Обновление чарта pro-api #548

merged 9 commits into from
Dec 26, 2024

Conversation

v-loboda
Copy link
Collaborator

@v-loboda v-loboda commented Dec 9, 2024

Pull Request description

Changelog

  • Some changes (A description of the changes proposed in the pull request.)

Issues

  • Issue-123 (Link to related issue)

Breaking changes

  • If there are breaking changes, they need to be added to the file Breaking-Changes

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 December 9, 2024 06:16
@v-loboda v-loboda requested a review from mount-li December 9, 2024 06:17
@v-loboda v-loboda marked this pull request as draft December 9, 2024 06:17
@v-loboda v-loboda removed the request for review from mount-li December 9, 2024 06:18
@v-loboda v-loboda force-pushed the PRO-6515-UpdateProApi branch 2 times, most recently from 777aa5f to f70077e Compare December 16, 2024 07:12
@v-loboda v-loboda marked this pull request as ready for review December 16, 2024 07:13
@v-loboda v-loboda force-pushed the PRO-6515-UpdateProApi branch from f70077e to 992ce73 Compare December 16, 2024 07:17
mprudnikov3
mprudnikov3 reviewed Dec 20, 2024
View reviewed changes
charts/pro-api/values.yaml Outdated Show resolved Hide resolved
charts/pro-api/README.md Outdated Show resolved Hide resolved
charts/pro-api/templates/permissions-api/permissions-api-deployment.yaml Show resolved Hide resolved
charts/pro-api/values.yaml Show resolved Hide resolved
charts/pro-api/values.yaml Show resolved Hide resolved
charts/pro-api/values.yaml Show resolved Hide resolved
@mprudnikov3
Copy link
Contributor

@v-loboda, подскажи, пожалуйста, по tasks: вижу, что это планировщик задач(hangfire).
Какое предназначение его, какую функциональность несет? нужен он нам сейчас?

@v-loboda
Copy link
Collaborator Author

@v-loboda, подскажи, пожалуйста, по tasks: вижу, что это планировщик задач(hangfire). Какое предназначение его, какую функциональность несет? нужен он нам сейчас?

Это микросервис для выполнения задач: подготовка данных слоя, обработка сообщений из кафки, периодические сервисные джобы. В него в будущем переедут штуки, которые вертятся внутри основного АПИ сейчас.
Он нам конечно же нужен, без него скоро совсем ничего работать не будет.

@v-loboda v-loboda force-pushed the PRO-6515-UpdateProApi branch from ef9d853 to 9eb67a8 Compare December 25, 2024 11:10
@v-loboda v-loboda force-pushed the PRO-6515-UpdateProApi branch from 9eb67a8 to ae253e2 Compare December 25, 2024 11:17
@mprudnikov3
Copy link
Contributor

tasks закрыли skipом
с приездом зависящей от tasks функциональности -- возвращаем

@mprudnikov3 mprudnikov3 merged commit 7645df8 into develop Dec 26, 2024
2 checks passed
@mprudnikov3 mprudnikov3 deleted the PRO-6515-UpdateProApi branch December 26, 2024 06:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dbelyaev-nsk dbelyaev-nsk dbelyaev-nsk left review comments

@mprudnikov3 mprudnikov3 mprudnikov3 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@v-loboda @mprudnikov3 @dbelyaev-nsk