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

[EVERGIS] websocket-services должен указывать на backend.service.name. Required defaults #457

Merged
merged 3 commits into from
Oct 17, 2024

Conversation

v-a-v
Copy link
Contributor

@v-a-v v-a-v commented Jun 21, 2024

Pull Request description

Changelog

  • websocket-services должен указывать на backend.service.name
    Надо проверить что валидация ingress проходит без ошибок и что веб-сокет работает.

  • Обязательные дефолтные значения в пустую строку согласно styleguide

Issues

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-a-v v-a-v requested review from a team as code owners June 21, 2024 06:44
@v-a-v v-a-v changed the title [EVERGIS] websocket-services должен указывать на backend.service.name [EVERGIS] websocket-services должен указывать на backend.service.name. Required defaults Jun 21, 2024
@dbelyaev-nsk
Copy link
Contributor

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

@mprudnikov3 mprudnikov3 merged commit 7160161 into develop Oct 17, 2024
2 checks passed
@mprudnikov3 mprudnikov3 deleted the avoronkov--gis-platform-fix-websocket-name branch October 17, 2024 11:46
mprudnikov3 added a commit that referenced this pull request Nov 7, 2024
* fix pro-api/values.yaml doc (#432)

* [tiles-api] Upgrade to 4.54.0 (#428)

* Add missing default for enabled_modules->predictors in citylens-web configmap (#433)

* navi-router 6.17.0.8 -> 6.17.1.2 (#434)

* [PRO-5009] Changed helm/hooks. (#436)

* pro-ui: New external style manager config (#426)

* pro-ui: New external style manager config

* GH Actions readme checker workflow (#438)

* base wf

* fix

* update

* update

* update

* update

* update

* update

* clean

* clean

* minor documentation changes

---------

Co-authored-by: Dmitrii Moskalev <[email protected]>

* Gh actions readme checker changing name (#440)

* base wf

* fix

* update

* update

* update

* update

* update

* update

* clean

* clean

* minor documentation changes

* changing name

---------

Co-authored-by: Dmitrii Moskalev <[email protected]>

* PRO-UI 2.14.0 (#441)

* PRO-UI fix hooks (#443)

* [PRO-5181] Обновление pro-api до версии 1.22.0 и pro-ui до версии 2.15.1 (#435)

* [PRO-5181] Обновление pro-api до версии 1.17

* [PRO-5181] Поправлены замечания по ревью

* [PRO-5181] Поправлен url для catalog

* [PRO-5181] обновил образ до 1.22.0

* [PRO-5181] Обновление pro-api до версии 1.17

* [PRO-5181] Поправлены замечания по ревью

* [PRO-5181] Поправлен url для catalog

* [PRO-5181] обновил образ до 1.22.0

* Add new version for PRO-UI

* Fix readme

* Citylens 1.9.0 & custom CA support (#439)

* Added custom CA support

* Update version to 1.9.0

* Fix customCA documentation

* Update readme

* Fix broken templating

* Fix citylens-web configmap (#448)

* Gis-platform. Мелкие правки по дефолтным параметрам (#450)

* [license] Fixed issue with k8s ca automount (#444)

* WAPI-23444 custom ca twins api (#437)

* WAPI-23444 custom ca twins api

* WAPI-23444 custom ca twins api

* WAPI-23444 custom ca twins api

* Bump citylens to 1.10.0 (#453)

* Bump citylens to 1.10.0

* Complete renaming pro.url to pro.baseUrl

* PRO-5559: upgrade key ui to latest (#449)

* PRO-5559: upgrade key ui to latest

* WAPI-23482 Add configuration for Public API sign (#429)

* WAPI-23482 Add configuration for Public API sign

* impr

* disabled by default

* optional signkey in secret

* optional signkey in secret

* upd image version

* reset chart version

---------

Co-authored-by: Kirill Salnikov <[email protected]>

* Правки в чартах mapgl-js-api  и floors-api (#420)

* Add ability to add extra env vars to citylens-web deployment (#455)

* Add ability to add extra env vars to citylens-web deployment

* Fix missing end tag

* chart/generic-chart syncup (#452)

* pro-api serviceAccountOverride support (#456)

* changes

* update

* value description added

* readme update

* readme update

---------

Co-authored-by: Dmitrii Moskalev <[email protected]>

* [chart/navi-back] 7.23.0.5 release (#451)

* fix conflict

* [ONPREM-839] Add revisionHistoryLimit (#459)

* 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

* add revision history limit to navi-services (#458)

* 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

* Add github action with Mattermost Notify (#465)

* [DEVOPS-1178] add navi back freeroam (#447)

* [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]>

* [PRO-5488] Change-Helm for PRO-UI (#460)

* Change-Helm for PRO-UI

* Fix Readme

* add link to docs.2gis.com

---------

Co-authored-by: Igor Sheykin <[email protected]>

* Add new citylens-workers based worker & add Drivers asset-related topics (#466)

* Update Chart.yaml

* changelog update

* conflict trash fix

* update

* Chore: remove unused helmfile (#480)

* search-api: custom CA certificates support (#478)

* Проверить ПР search 478. Fix readme (#485)

* Обновление PRO API до версии 1.35.0 и PRO UI до версии 2.27.0 (#481)

* Citylens 1.12.0 (#484)

* Update Citylens 1.12.0
---------

Co-authored-by: Petr Beklemishev <[email protected]>

* chart/navi-back syncup (#470)

* Chore: styleguide kafka example (#488)

* WAPI-23606 catalog search config (#475)

* DEVOPS-1348: add bicycle nginx location to navi-front (#487)

* Platform 0.15.0 (#483)

* GEFEST-82: update platform to 0-15-0

* GEFEST-82: update pages comment

---------

Co-authored-by: Milov Dmitriy <[email protected]>

* TILES-6466 mapgl-js-api: add styles server vars (#469)

* mapgl-js-api: add styles server vars

* Apply suggestions from code review

Co-authored-by: Michel Beloshitsky <[email protected]>

* Link readme

* Use mapgl-api 1.50.0

* Update mapgl

---------

Co-authored-by: Нищета Дмитрий Антонович <[email protected]>
Co-authored-by: Michel Beloshitsky <[email protected]>
Co-authored-by: Igor Sheykin <[email protected]>

* WAPI-23337 Интеграция catalog api с сервисом license v2 (#486)

* fix pro-ui values

* [navi-async-matrix] netloc must start with scheme (#477)

* [chart/navi-back] chart syncup (#493)

* Upgrade version of platform UI to 0.15.1 (#491)

* Mapgl styles default urls fix (#494)

* style urls fix
* update default values and description
* update readme
* defined mapgl_demo_key

* [keys-api] empty default values (#499)

* WAPI-23578 Styles API (#482)

* WAPI-23578 Styles API

* WAPI-23578 Remove configurable s3 bucket for styles

* Revert "WAPI-23578 Remove configurable s3 bucket for styles"

This reverts commit f7f3e6f.

* Styles API version as 1.0.0

* Add missiong secrets deploy annotations

* Increase ingress default body size, remove useless worker disable

* Fix ingress values annotations

* Disable ingress nginx params

* Use secret in jobs deploy

* Worker as deployment

* Set default nginx ingress body size

* Set actual Styles API release version

* Fix notes Twins mension

* fix initialDelaySeconds for worker

* No useless peristent volume

* rename endpoint to host, update descriptions

* fix required

* fix readme

---------

Co-authored-by: Gopanenko Pavel <[email protected]>
Co-authored-by: Igor Sheykin <[email protected]>

* [DEVOPS-1394] fix typo in castle chart (#501)

* Release 1.28.2 (#505) (#507)

* Change variables naming (#506)

* [stat-receiver] Change variables naming


---------

Co-authored-by: Sergey Vlasov <[email protected]>

* add required field to mapgl ENV values (#504)

* Citylens 1.13.0 (#508)

* DEVOPS-1409: add securityContext (#510)

* [license] Upgraded app version to 2.2.3 (#496)

* [license] Upgraded app version to 2.2.2

* [license] Upgraded app version to 2.2.3 and added dgctlStorage.region, persistence.region to values

* Изменение структуры чарта pro-api (#490)

* Improve MapGL JS API params (#509)

* [chart/navi-back] chart syncup (#498)

* Added custom CA support

---------

Co-authored-by: Morozov Dmitriy <[email protected]>

* add custom ca for navi-castle, navi-restrictions, navi-async-matrix (#511)

Co-authored-by: Morozov Dmitriy <[email protected]>

* sm fix

* Fix customCAs include in navi-castle cronjob template

* Revert "Fix customCAs include in navi-castle cronjob template"

This reverts commit 9db512e.

* Add github action "MM Release notification" (#516)

* [ONPREM-932] test notif release mm

* [ONPREM-932] change tests rules

* [ONPREM-932] add changes job rules

* [ONPREM-932] fix on.workflows

* [ONPREM-932] fixes style

* [ONPREM-932] rename PR notif action & fix release notif

---------

Co-authored-by: mprudnikov3 <[email protected]>

* [gis-platform] Do not use config template (#512)

* [EVERGIS] websocket-services должен указывать на backend.service.name. Required defaults (#457)

* [EVERGIS] websocket-services должен указывать на backend.service.name

* [EVERGIS] websocket-services должен указывать на backend.service.name. Default values

* [EVERGIS] websocket-services должен указывать на backend.service.name. Default values

---------

Co-authored-by: Alexander Voronkov <[email protected]>

* [traffic-proxy] Update traffic-proxy chart configuration and nginx template (#518)

* Update traffic-proxy chart configuration and nginx template

* Add customizable server section

* Update charts/traffic-proxy/templates/configmap.yaml

Co-authored-by: Voronkov Alexander <[email protected]>

* Update charts/traffic-proxy/templates/configmap.yaml

Co-authored-by: Voronkov Alexander <[email protected]>

* Change log configuration

* Change README

* lint fix

* Delete duplicate field description in traffic-proxy

* Fix configmap spaces. Add loglevel field allowed values

---------

Co-authored-by: Sergey Vlasov <[email protected]>
Co-authored-by: Voronkov Alexander <[email protected]>

* Upgrade to 4.61.0 (#514)

Co-authored-by: p.fomin <[email protected]>
Co-authored-by: mprudnikov3 <[email protected]>

* [PRO-6241] Обновление PRO API до версии 1.41.1 (#513)

* [GEFEST-877] Dispatcher deployment and cleaner cronjob (#446)

* Chore: Update styleguide for logging block (#521)

* Citylens 1.14.0 (#523)

* Release 1.30.0 (#525)

Co-authored-by: mprudnikov <[email protected]>

* Release 1.30.0 (#526) (#527)

* Fix lint warning for accessLog value (#529)

* TILES-6894 floors-api: add user certs (#524)

* floors-api: add user certs

* Format

---------

Co-authored-by: Нищета Дмитрий Антонович <[email protected]>

* Bump mapgl-js-api (#530)

* Fix release MM notification  (#528)

* Fix customCAs include in navi-castle cronjob template (#517)

* WAPI-23658 добавлены недостающие параметры в конфиг dgctlStorage (#503)

* WAPI-23658 добавлены недостающие параметры в конфиг dgctlStorage

* WAPI-23658 добавлены недостающие параметры в конфиг dgctlStorage

* WAPI-23658 добавлены недостающие параметры в конфиг dgctlStorage

* WAPI-23658 добавлены недостающие параметры в конфиг dgctlStorage

---------

Co-authored-by: Donov Dmitriy <[email protected]>

* chart/navi-back syncup (#531)

* release 1.31.0

---------

Co-authored-by: Igor Sheykin <[email protected]>
Co-authored-by: FreakyGranny <[email protected]>
Co-authored-by: Петр Беклемишев <[email protected]>
Co-authored-by: i-bogomazov <[email protected]>
Co-authored-by: Andrew Mikhailov <[email protected]>
Co-authored-by: Konstantin Lobov <[email protected]>
Co-authored-by: DAMoskalev <[email protected]>
Co-authored-by: Dmitrii Moskalev <[email protected]>
Co-authored-by: Aveldin1 <[email protected]>
Co-authored-by: Денис Беляев <[email protected]>
Co-authored-by: Denis Belyaev <[email protected]>
Co-authored-by: v-loboda <[email protected]>
Co-authored-by: Sergey Ordin <[email protected]>
Co-authored-by: Voronkov Alexander <[email protected]>
Co-authored-by: Vladimir Popov <[email protected]>
Co-authored-by: Dmitriy Donov <[email protected]>
Co-authored-by: ostrovskiy2gis <[email protected]>
Co-authored-by: Kirill Salnikov <[email protected]>
Co-authored-by: Michel Beloshitsky <[email protected]>
Co-authored-by: Полозов Игорь Сергеевич <[email protected]>
Co-authored-by: uk-navi-ci <[email protected]>
Co-authored-by: Sergey Vlasov <[email protected]>
Co-authored-by: Alexandr Sentyabov <[email protected]>
Co-authored-by: Сентябов Александр Сергеевич <[email protected]>
Co-authored-by: Igor Sheykin <[email protected]>
Co-authored-by: Andrey Morozov <[email protected]>
Co-authored-by: zhukov-d-man <[email protected]>
Co-authored-by: Dmitrii Molochnikov <[email protected]>
Co-authored-by: Petr Beklemishev <[email protected]>
Co-authored-by: Morozov Dmitriy <[email protected]>
Co-authored-by: Dmitry Milov <[email protected]>
Co-authored-by: Milov Dmitriy <[email protected]>
Co-authored-by: Dmitry Nischeta <[email protected]>
Co-authored-by: Нищета Дмитрий Антонович <[email protected]>
Co-authored-by: vgivanov <[email protected]>
Co-authored-by: Pavel Gopanenko <[email protected]>
Co-authored-by: Gopanenko Pavel <[email protected]>
Co-authored-by: Sergey Vlasov <[email protected]>
Co-authored-by: Morozov Dmitriy <[email protected]>
Co-authored-by: mprudnikov3 <[email protected]>
Co-authored-by: Alexander Voronkov <[email protected]>
Co-authored-by: p.fomin <[email protected]>
Co-authored-by: ArtemPolykov <[email protected]>
Co-authored-by: urbiae <[email protected]>
Co-authored-by: Donov Dmitriy <[email protected]>
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.

4 participants