big-search #794
big-search #794
Conversation
| apiVersion: v2 | ||
| name: search-api | ||
| description: Search engine for catalog | ||
| name: big-search |
There was a problem hiding this comment.
раз уж заменяем чарт, тогда так
| name: big-search | |
| name: search-api |
There was a problem hiding this comment.
Всё таки пришли к такому варианту. Поправлю.
| # incremented each time you make changes to the application. Versions are not expected to | ||
| # follow Semantic Versioning. They should reflect the version the application is using. | ||
| # It is recommended to use it with quotes. | ||
| appVersion: "8.14.6" |
There was a problem hiding this comment.
| appVersion: "8.14.6" | |
| appVersion: 8.14.6 |
| @@ -0,0 +1,301 @@ | |||
| # bluegreen: 'blue' | |||
There was a problem hiding this comment.
это же временный файл? потом уберется?
There was a problem hiding this comment.
Это пример с которым я деплоил, который учитывает ваши требования.
Он ужен вам как минимум для первого деплоя, далее могу выпилить.
| repository: docker-hub.2gis.ru/usrch/bs-worker | ||
| pullPolicy: IfNotPresent | ||
| tag: 0.1 |
There was a problem hiding this comment.
имя образа должно отражать имя сервиса, к которому относится и тэги должны быть в semver
| repository: docker-hub.2gis.ru/usrch/bs-worker | |
| pullPolicy: IfNotPresent | |
| tag: 0.1 | |
| repository: 2gis-on-premise/search-worker | |
| pullPolicy: IfNotPresent | |
| tag: 0.1.0 |
There was a problem hiding this comment.
окей переделаю
| loggerLevel: "info" | ||
| loggerFormat: "json" |
There was a problem hiding this comment.
дефолтные строки без кавычек
| loggerLevel: "info" | |
| loggerFormat: "json" | |
| loggerLevel: info | |
| loggerFormat: json |
There was a problem hiding this comment.
Никаких кавычек, понял.
| podSecurityContext: {} | ||
| securityContext: {} | ||
| hpa: | ||
| enabled: true |
There was a problem hiding this comment.
hpa, vpa по дефолту отключаем
| enabled: true | |
| enabled: false |
| enabled: true | ||
| updateMode: "Off" |
There was a problem hiding this comment.
| enabled: true | |
| updateMode: "Off" | |
| enabled: false | |
| updateMode: Off |
|
|
||
| ingress: | ||
| enabled: false | ||
| className: "nginx" |
There was a problem hiding this comment.
| className: "nginx" | |
| className: nginx |
|
|
||
| # @param onpremise Enable specific parameters for on-premise deployment | ||
|
|
||
| onpremise: false |
There was a problem hiding this comment.
| onpremise: false | |
| onpremise: true |
| bucket: '' | ||
| accessKey: '' | ||
| secretKey: '' | ||
| manifest: '1' |
There was a problem hiding this comment.
| manifest: '1' | |
| manifest: '' |
2 participants
Pull Request description
Changelog
Большой поиск в On-Prem(Полностью новый поиск)Issues
Issue-123(Link to related issue)Breaking changes
Это полностью новое приложение, не знаю на сколько это Breaking-Changes, на 0 или на 100%.
Check-list. Чек-лист код-ревью
helm lint charts/search-api