-
Notifications
You must be signed in to change notification settings - Fork 917
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
[Manual backport 2.x] Add Dynamic Config Service to Core Service (#7194) #7958
[Manual backport 2.x] Add Dynamic Config Service to Core Service (#7194) #7958
Conversation
* Add dynamic config service to core Signed-off-by: Huy Nguyen <[email protected]> * Add opensearch client implementation Signed-off-by: Huy Nguyen <[email protected]> * Add descriptions for the DAO clients Signed-off-by: Huy Nguyen <[email protected]> * Refactor DynamicConfigService Signed-off-by: Huy Nguyen <[email protected]> * Refactor dynamic config service start Signed-off-by: Huy Nguyen <[email protected]> --------- Signed-off-by: Huy Nguyen <[email protected]> (cherry picked from commit 8a96664) Fix issue when bootstrapping on 2.x (opensearch-project#7901) * Fix issue bootstrapping dynamic config service Signed-off-by: Huy Nguyen <[email protected]> * Changeset file for PR opensearch-project#7901 created/updated --------- Signed-off-by: Huy Nguyen <[email protected]> Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com> (cherry picked from commit 07504f6) Fix mock import (opensearch-project#7922) Signed-off-by: Huy Nguyen <[email protected]> (cherry picked from commit b370871)
❌ Empty Changelog SectionThe Changelog section in your PR description is empty. Please add a valid changelog entry or entries. If you did add a changelog entry, check to make sure that it was not accidentally included inside the comment block in the Changelog section. |
Signed-off-by: Huy Nguyen <[email protected]>
❌ Empty Changelog SectionThe Changelog section in your PR description is empty. Please add a valid changelog entry or entries. If you did add a changelog entry, check to make sure that it was not accidentally included inside the comment block in the Changelog section. |
❌ Empty Changelog SectionThe Changelog section in your PR description is empty. Please add a valid changelog entry or entries. If you did add a changelog entry, check to make sure that it was not accidentally included inside the comment block in the Changelog section. |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## 2.x #7958 +/- ##
==========================================
+ Coverage 61.01% 61.06% +0.04%
==========================================
Files 3670 3682 +12
Lines 86748 87024 +276
Branches 13332 13364 +32
==========================================
+ Hits 52931 53137 +206
- Misses 30621 30681 +60
- Partials 3196 3206 +10
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Manual backports of #7194 #7901 and #7922
Add Dynamic Config Service to Core Service (#7194)
Signed-off-by: Huy Nguyen [email protected]
Signed-off-by: Huy Nguyen [email protected]
Signed-off-by: Huy Nguyen [email protected]
Signed-off-by: Huy Nguyen [email protected]
Signed-off-by: Huy Nguyen [email protected]
Signed-off-by: Huy Nguyen [email protected]
(cherry picked from commit 8a96664)
Fix issue when bootstrapping on 2.x (#7901)
Signed-off-by: Huy Nguyen [email protected]
Signed-off-by: Huy Nguyen [email protected]
Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com>
(cherry picked from commit 07504f6)
Fix mock import (#7922)
Signed-off-by: Huy Nguyen [email protected]
(cherry picked from commit b370871)
Description
Issues Resolved
Screenshot
Testing the changes
Changelog
Check List
yarn test:jest
yarn test:jest_integration