-
Notifications
You must be signed in to change notification settings - Fork 578
Updating deprecated options #9602
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
base: main
Are you sure you want to change the base?
Conversation
Thank you for submitting your PR. The PR states are In progress (or Draft) -> Tech review -> Doc review -> Editorial review -> Merged. Before you submit your PR for doc review, make sure the content is technically accurate. If you need help finding a tech reviewer, tag a maintainer. When you're ready for doc review, tag the assignee of this PR. The doc reviewer may push edits to the PR directly or leave comments and editorial suggestions for you to address (let us know in a comment if you have a preference). The doc reviewer will arrange for an editorial review. |
@landon-lengyel Could you please resolve DCO? |
@cwperks Could you please review this PR? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@landon-lengyel These settings still exist in the security plugin and are not marked as deprecated.
I see logic in the security plugin that either configuration is permissible.
Are you experiencing any errors configuring with the documented settings?
@kolchfa-aws I'm sorry, I'm not sure what to do for the DCO. I did include |
@reta I see you have the security plugin settings marked as deprecated in opensearch-project/security@e2a06f0. Is there a plan to remove them? The options I see are 1) remove the deprecated setting or 2) undeprecate the setting and support with both the |
@cwperks It looks like you were able to answer your own question, but to clarify yes I am seeing warnings in the deprecation logs. Such as:
There's a bit more info about that on security issue 4780 |
@landon-lengyel To fix DCO, rebase the branch as described here https://github.com/opensearch-project/documentation-website/pull/9602/checks?check_run_id=40407368811. |
Oh yeah, thanks @cwperks , I think those all are in core (netty4 transport module) now, we could removed then in 3.0.0, thank you |
Signed-off-by: Landon Lengyel <landon@almonde.org> Signed-off-by: Landon Lengyel <landon.lengyel@slcschools.org> Signed-off-by: Landon Lengyel <landon@almonde.org>
@landon-lengyel Do you still plan to proceed with this PR? |
Description
Updates options that are marked as deprecated.
Issues Resolved
Resolves issue #9466 if these two options are all that's changed.
Version
v2.19.1
Frontend features
n/a
Checklist
For more information on following Developer Certificate of Origin and signing off your commits, please check here.