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

add required field to mapgl ENV values #504

Merged
merged 5 commits into from
Sep 23, 2024

Conversation

Ashigo
Copy link
Contributor

@Ashigo Ashigo commented Sep 18, 2024

Pull Request description

Changelog

  • add required fields to two mapgl ENVs

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

@Ashigo Ashigo requested review from a team as code owners September 18, 2024 06:41
@Ashigo Ashigo changed the title add required to values add required field to mapgl ENVvalues Sep 18, 2024
@Ashigo Ashigo changed the title add required field to mapgl ENVvalues add required field to mapgl ENV values Sep 18, 2024
@endryhold endryhold merged commit 01e0d09 into develop Sep 23, 2024
2 checks passed
@endryhold endryhold deleted the feature/ONPREM-1078-fix-required-envs branch September 23, 2024 10:44
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 <d.moskalev@2gis.ru>

* 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 <d.moskalev@2gis.ru>

* 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 <k.salnikov@2gis.ru>

* Правки в чартах 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 <d.moskalev@2gis.ru>

* [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 <mora378@rambler.ru>
Co-authored-by: Aveldin1 <164148461+Aveldin1@users.noreply.github.com>
Co-authored-by: Сентябов Александр Сергеевич <a.sentyabov@2gis.ru>

* [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 <sheikin.igor@gmail.com>

* 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 <p.beklemishev@2gis.ru>

* 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 <d.milov@2gis.ru>

* 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 <itano@yandex.ru>

* Link readme

* Use mapgl-api 1.50.0

* Update mapgl

---------

Co-authored-by: Нищета Дмитрий Антонович <d.nishcheta@2gis.ru>
Co-authored-by: Michel Beloshitsky <itano@yandex.ru>
Co-authored-by: Igor Sheykin <49325415+Ashigo@users.noreply.github.com>

* 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 <p.gopanenko@2gis.ru>
Co-authored-by: Igor Sheykin <sheikin.igor@gmail.com>

* [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 <s.vlasov@2gis.ru>

* 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 <dm.morozov@2gis.ru>

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

Co-authored-by: Morozov Dmitriy <dm.morozov@2gis.ru>

* 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 <m.prudnikov@2gis.ru>

* [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 <a.voronkov@2gis.ru>

* [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 <voronkov.alexander@gmail.com>

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

Co-authored-by: Voronkov Alexander <voronkov.alexander@gmail.com>

* 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 <s.vlasov@2gis.ru>
Co-authored-by: Voronkov Alexander <voronkov.alexander@gmail.com>

* Upgrade to 4.61.0 (#514)

Co-authored-by: p.fomin <p.fomin@2gis.ru>
Co-authored-by: mprudnikov3 <139206622+mprudnikov3@users.noreply.github.com>

* [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 <m.prudnikov@2gis.ru>

* 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: Нищета Дмитрий Антонович <d.nishcheta@2gis.ru>

* 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 <d.donov@2gis.ru>

* chart/navi-back syncup (#531)

* release 1.31.0

---------

Co-authored-by: Igor Sheykin <49325415+Ashigo@users.noreply.github.com>
Co-authored-by: FreakyGranny <mora378@rambler.ru>
Co-authored-by: Петр Беклемишев <pbekl@bk.ru>
Co-authored-by: i-bogomazov <106957509+i-bogomazov@users.noreply.github.com>
Co-authored-by: Andrew Mikhailov <Andrew.Mikhailov.18.01@gmail.com>
Co-authored-by: Konstantin Lobov <lobovke@gmail.com>
Co-authored-by: DAMoskalev <33502773+DAMoskalev@users.noreply.github.com>
Co-authored-by: Dmitrii Moskalev <d.moskalev@2gis.ru>
Co-authored-by: Aveldin1 <164148461+Aveldin1@users.noreply.github.com>
Co-authored-by: Денис Беляев <d.belyaev@2gis.ru>
Co-authored-by: Denis Belyaev <74010851+dbelyaev-nsk@users.noreply.github.com>
Co-authored-by: v-loboda <111125664+v-loboda@users.noreply.github.com>
Co-authored-by: Sergey Ordin <61708156+sergeLeLe@users.noreply.github.com>
Co-authored-by: Voronkov Alexander <voronkov.alexander@gmail.com>
Co-authored-by: Vladimir Popov <v.popov@2gis.ru>
Co-authored-by: Dmitriy Donov <dmitriy@donov.ru>
Co-authored-by: ostrovskiy2gis <108522609+ostrovskiy2gis@users.noreply.github.com>
Co-authored-by: Kirill Salnikov <k.salnikov@2gis.ru>
Co-authored-by: Michel Beloshitsky <itano@yandex.ru>
Co-authored-by: Полозов Игорь Сергеевич <i.polozov@2gis.ru>
Co-authored-by: uk-navi-ci <161344134+uk-navi-ci@users.noreply.github.com>
Co-authored-by: Sergey Vlasov <sergeys.vlasov@gmail.com>
Co-authored-by: Alexandr Sentyabov <36183816+SanchezzRU@users.noreply.github.com>
Co-authored-by: Сентябов Александр Сергеевич <a.sentyabov@2gis.ru>
Co-authored-by: Igor Sheykin <sheikin.igor@gmail.com>
Co-authored-by: Andrey Morozov <62840181+endryhold@users.noreply.github.com>
Co-authored-by: zhukov-d-man <zhukov.d@2gis.ru>
Co-authored-by: Dmitrii Molochnikov <d.molochnikov@2gis.ru>
Co-authored-by: Petr Beklemishev <p.beklemishev@2gis.ru>
Co-authored-by: Morozov Dmitriy <hacker000@yandex.ru>
Co-authored-by: Dmitry Milov <senneerr@gmail.com>
Co-authored-by: Milov Dmitriy <d.milov@2gis.ru>
Co-authored-by: Dmitry Nischeta <55282960+dnischeta@users.noreply.github.com>
Co-authored-by: Нищета Дмитрий Антонович <d.nishcheta@2gis.ru>
Co-authored-by: vgivanov <v.ivanov@2gis.ru>
Co-authored-by: Pavel Gopanenko <pavelgopanenko@gmail.com>
Co-authored-by: Gopanenko Pavel <p.gopanenko@2gis.ru>
Co-authored-by: Sergey Vlasov <s.vlasov@2gis.ru>
Co-authored-by: Morozov Dmitriy <dm.morozov@2gis.ru>
Co-authored-by: mprudnikov3 <m.prudnikov@2gis.ru>
Co-authored-by: Alexander Voronkov <a.voronkov@2gis.ru>
Co-authored-by: p.fomin <p.fomin@2gis.ru>
Co-authored-by: ArtemPolykov <44114018+ArtemPolykov@users.noreply.github.com>
Co-authored-by: urbiae <156092383+urbiae@users.noreply.github.com>
Co-authored-by: Donov Dmitriy <d.donov@2gis.ru>
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