-
Notifications
You must be signed in to change notification settings - Fork 280
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.x] XContent Refactor #2598
[Backport 2.x] XContent Refactor #2598
Conversation
Signed-off-by: Ryan Liang <jiallian@amazon.com>
Signed-off-by: Ryan Liang <jiallian@amazon.com>
I will look into the test failure introduced by |
I'm updating my findings here:
It shows like there is a communication mismatch. I decoded the record, and it gives a
I think by enabling the openSSL, we need a |
Signed-off-by: Ryan Liang <jiallian@amazon.com>
a7d7a8e
to
6575a27
Compare
I've backported the ClusterManager changes into my fork. Assuming the test pass https://github.com/peternied/security/actions/runs/4569414539/jobs/8065642541 I think we should merge that into this branch to move forward. See original source PR What do you think @RyanL1997? |
Hi, @peternied, |
Signed-off-by: Ryan Liang <jiallian@amazon.com>
Signed-off-by: Ryan Liang <jiallian@amazon.com>
* Cluster manager inclusive checks on codebase Enforce usage of ClusterManager terminology in the codebase Include mechanism to disable checkstyle rule Cross cluster needs to have an additional role in the `node.roles` list, which I am guessing was backwards compatiable if the legacy versions of the role assignments were used. With this change cross cluster tests properly include this value during setup and the settings for these values are merged instead of being overridden. Signed-off-by: Peter Nied <petern@amazon.com>
Codecov Report
📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more @@ Coverage Diff @@
## 2.x #2598 +/- ##
============================================
- Coverage 61.30% 61.19% -0.11%
+ Complexity 3314 3305 -9
============================================
Files 258 258
Lines 18272 18272
Branches 3251 3251
============================================
- Hits 11201 11182 -19
- Misses 5496 5508 +12
- Partials 1575 1582 +7
... and 7 files with indirect coverage changes Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
Description
Reacts to refactoring done on xcontent classes from core. Many imports from org.opensearch.common.xcontent were switched to org.opensearch.core.xcontent.
Build fix
Issues Resolved
Check List
New functionality includes testingNew functionality has been documentedBy submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.