YDBOPS-9679 dynconfig CMS ReplaceConfig - Githubissues

ydb-platform / ydb-kubernetes-operator

YDB Operator allows you to deploy your own YDB cluster in Kubernetes

Other

47 stars 18 forks source link

YDBOPS-9679 dynconfig CMS ReplaceConfig #214

Closed kobzonega closed 1 month ago

kobzonega commented 5 months ago

I hereby agree to the terms of the CLA available at: https://yandex.ru/legal/cla/?lang=en

Pull request type

Please check the type of change your PR introduces:

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] Documentation content changes
[ ] Other (please describe):

What is the current behavior?

Issue Number: YDBOPS-9679

What is the new behavior?

validate dynconfig for kubernetes (must contain mandatory blobstorage fields)
insert .config field without changes from dynconfig into ConfigMap as static config
check the need for ReplaceConfig by compare metadata.version with CMS GetConfig request
update config in cluster by send dynconfig with CMS ReplaceConfig request

Other information

nmerkulov commented 4 months ago

few nits

overall lgtm