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

WIP: catalog api bss #258

Open
wants to merge 12 commits into
base: develop
Choose a base branch
from
Open

WIP: catalog api bss #258

wants to merge 12 commits into from

Conversation

DemonNSK
Copy link
Contributor

@DemonNSK DemonNSK commented Apr 21, 2023

Pull Request description

Changelog

  • Отправка bss сообщений двух типов (request и search) из catalog api

Issues

Breaking changes

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

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

charts/catalog-api/values.yaml Outdated Show resolved Hide resolved
charts/catalog-api/values.yaml Outdated Show resolved Hide resolved
charts/catalog-api/values.yaml Outdated Show resolved Hide resolved
charts/catalog-api/values.yaml Outdated Show resolved Hide resolved
charts/catalog-api/values.yaml Outdated Show resolved Hide resolved
@pavelgopanenko
Copy link
Collaborator

👍 LGTM

@vgivanov
Copy link
Contributor

В онпреме вместо bss используется название stat-receiver. Так что тут лучше бы переименовать, можно в просто stat наверное.

@Lolimpo
Copy link
Collaborator

Lolimpo commented Jul 10, 2023

В онпреме вместо bss используется название stat-receiver. Так что тут лучше бы переименовать, можно в просто stat наверное.

@vgivanov Привет, подскажи, пожалуйста, у нави сейчас наблюдаю bss. Оставим однообразно все-таки может сейчас? Чтобы глаза не разбегались от stat/bss/reciever'ов. Нам в целом без разницы, можем у себя сейчас переименовать, лишь бы было прозрачно что за что отвечает и путаница вдруг у партнеров не возникла отчего в одном сервисе stat-receiver, а чего в другом bss какой-то и откуда их всех брать.

@Lolimpo Lolimpo changed the title WIP: catalog api bss catalog api bss Jul 26, 2023
@salkrr salkrr changed the title catalog api bss WIP: catalog api bss Jul 29, 2023
@DemonNSK DemonNSK changed the title WIP: catalog api bss catalog api bss Aug 3, 2023
@DemonNSK DemonNSK requested review from a team as code owners February 20, 2024 11:57
@DemonNSK DemonNSK changed the base branch from master to develop February 20, 2024 11:59
@DemonNSK
Copy link
Contributor Author

В онпреме вместо bss используется название stat-receiver. Так что тут лучше бы переименовать, можно в просто stat наверное.

@vgivanov Привет, подскажи, пожалуйста, у нави сейчас наблюдаю bss. Оставим однообразно все-таки может сейчас? Чтобы глаза не разбегались от stat/bss/reciever'ов. Нам в целом без разницы, можем у себя сейчас переименовать, лишь бы было прозрачно что за что отвечает и путаница вдруг у партнеров не возникла отчего в одном сервисе stat-receiver, а чего в другом bss какой-то и откуда их всех брать.

переименовал в stat и актуализировал, давайте уже мерджить :)

@endryhold
Copy link
Contributor

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

@DAMoskalev
Copy link
Contributor

В develop добавлена дополнительная проверка актуальности README файлов. Необходимо подмержить develop в текущую ветку. После добавления кода из develop автоматически запустится валидация readme файлов

# Conflicts:
#	charts/catalog-api/templates/api/deployment.yaml
@DemonNSK
Copy link
Contributor Author

В develop добавлена дополнительная проверка актуальности README файлов. Необходимо подмержить develop в текущую ветку. После добавления кода из develop автоматически запустится валидация readme файлов

подмержил

@DemonNSK DemonNSK changed the title catalog api bss WIP: catalog api bss Aug 29, 2024
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.

7 participants