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

WIP: catalog api bss #258

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

WIP: catalog api bss #258

wants to merge 59 commits into from

Conversation

DemonNSK
Copy link
Contributor

@DemonNSK DemonNSK commented Apr 21, 2023 •
edited
Loading

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

salkrr
salkrr reviewed Apr 25, 2023
View reviewed changes
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

Ashigo and others added 8 commits May 6, 2024 16:36
@Ashigo
fix pro-api/values.yaml doc (#432)
1724b74
@FreakyGranny
[tiles-api] Upgrade to 4.54.0 (#428)
b076674
@PeterBeklemishev
Add missing default for enabled_modules->predictors in citylens-web c…
5e0f53a 
…onfigmap (#433)
@i-bogomazov
navi-router 6.17.0.8 -> 6.17.1.2 (#434)
5fd2f18
@m7v
[PRO-5009] Changed helm/hooks. (#436)
07e4068
@kos33rd
pro-ui: New external style manager config (#426)
ec3cfca 
* pro-ui: New external style manager config
@DAMoskalev
GH Actions readme checker workflow (#438)
79bc905 
* base wf

* fix

* update

* update

* update

* update

* update

* update

* clean

* clean

* minor documentation changes

---------

Co-authored-by: Dmitrii Moskalev <[email protected]>
@DAMoskalev
Gh actions readme checker changing name (#440)
dc62228 
* base wf

* fix

* update

* update

* update

* update

* update

* update

* clean

* clean

* minor documentation changes

* changing name

---------

Co-authored-by: Dmitrii Moskalev <[email protected]>
i-bogomazov and others added 26 commits June 25, 2024 11:58
@i-bogomazov
chart/generic-chart syncup (#452)
2b50f21
@DAMoskalev
pro-api serviceAccountOverride support (#456)
5785840 
* changes

* update

* value description added

* readme update

* readme update

---------

Co-authored-by: Dmitrii Moskalev <[email protected]>
@uk-navi-ci
[chart/navi-back] 7.23.0.5 release (#451)
1d56357
Merge master
0294050
fix conflict
5871c13
@Ashigo
[ONPREM-839] Add revisionHistoryLimit (#459)
ebec6e6 
* add revisionHistoryLimit spec for catalog-api

* add revisionHistoryLimit for citylens

* add revisionHistoryLimit for dgtt

* add revisionHistoryLimit for floors-api

* add revisionHistoryLimit to gis-platform

* add revisionHistoryLimit to keycloak

* add revisionHistoryLimit to kays

* add revisionHistoryLimit to license

* add revisionHistoryLimit to mapgl

* add revisionHistoryLimit to platform

* add revisionHistoryLimit to search-api

* add revisionHistoryLimit to stat-reciever

* add revisionHistoryLimit to traffic-proxy

* Add revisionHistoryLimit to twins-api

* update readme files

* fix schema and value path

* fix license lint error

* remove revisionHistoryLimit for StatefulSet
@Ashigo
add revision history limit to navi-services (#458)
391e6b5 
* add revisionHistoryLimit to navi-async-matrix

* add revisionHistoryLimit to navi-castle

* add revisionHistoryLimit to navi-front

* add revisionHistoryLimit to navi-restrictions

* add revisionHistoryLimit to navi-router

* update readme files for navi

* update default value to 3

* remove revisionHistoryLimit for StatefulSet
@Efber
Add github action with Mattermost Notify (#465)
5441daa
@SanchezzRU @FreakyGranny @Aveldin1
[DEVOPS-1178] add navi back freeroam (#447)
9af0681 
* [tiles-api] Upgrade to 4.54.0 (#428)

* PRO-UI 2.14.0 (#441)

* [DEVOPS-1178] add navi back freeroam

* [DEVOPS-1178] up versions and add rules for free roam

* [DEVOPS-1178] typo

* [DEVOPS-1178] add gzip

---------

Co-authored-by: FreakyGranny <[email protected]>
Co-authored-by: Aveldin1 <[email protected]>
Co-authored-by: Сентябов Александр Сергеевич <[email protected]>
@Aveldin1 @Ashigo
[PRO-5488] Change-Helm for PRO-UI (#460)
781e696 
* Change-Helm for PRO-UI

* Fix Readme

* add link to docs.2gis.com

---------

Co-authored-by: Igor Sheykin <[email protected]>
@PeterBeklemishev
Add new citylens-workers based worker & add Drivers asset-related top…
34701cc 
…ics (#466)
Merge master
8790dee
@DAMoskalev
Update Chart.yaml
6d2fb86
changelog update
866ddfa
conflict trash fix
ca3520b
update
9a91285
Merge branch 'master' into develop
0ddfc8f
@endryhold
Chore: remove unused helmfile (#480)
0bb2be5
@zhukov-d-man
search-api: custom CA certificates support (#478)
576873d
@v-a-v
Проверить ПР search 478. Fix readme (#485)
89d0660
@dmolochnikov
Обновление PRO API до версии 1.35.0 и PRO UI до версии 2.27.0 (#481)
8a561ae
@PeterBeklemishev
Citylens 1.12.0 (#484)
cf78afb 
* Update Citylens 1.12.0
---------

Co-authored-by: Petr Beklemishev <[email protected]>
@uk-navi-ci
chart/navi-back syncup (#470)
1952c4b
@endryhold
Chore: styleguide kafka example (#488)
bb112bd
@DemonNSK
WAPI-23606 catalog search config (#475)
d751a56
Merge remote-tracking branch 'origin/develop' into catalog-api-bss
dff1ec4
@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
Reviewers

@salkrr salkrr salkrr 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.