-
Notifications
You must be signed in to change notification settings - Fork 967
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
[Backport 2.16] [MDS] Fix the hide local cluster config #7507
Conversation
* Fix the local cluster config Signed-off-by: Ryan Liang <jiallian@amazon.com> * Fix the local cluster config and add test Signed-off-by: Ryan Liang <jiallian@amazon.com> * Changeset file for PR #7497 created/updated --------- Signed-off-by: Ryan Liang <jiallian@amazon.com> Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com> (cherry picked from commit e24e535) Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
❌ Invalid Changelog HeadingThe '## Changelog' heading in your PR description is either missing or malformed. Please make sure that your PR description includes a '## Changelog' heading with proper spelling, capitalization, spacing, and Markdown syntax. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## 2.16 #7507 +/- ##
=======================================
Coverage 64.21% 64.21%
=======================================
Files 3614 3614
Lines 78750 78750
Branches 12446 12446
=======================================
+ Hits 50567 50569 +2
Misses 25131 25131
+ Partials 3052 3050 -2
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
the failing test is:
and it seems like it is not related to the change, and we may need a re-run |
right now im seeing the bootstrap failures:
|
Backport e24e535 from #7497.