Skip to content
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

Update references to default admin credentials #692

Closed
wants to merge 1 commit into from

Conversation

derek-ho
Copy link

@derek-ho derek-ho commented Dec 28, 2023

Description

Updates docs to inform about changes in security plugin in 2.12.0 that enforces an initial admin password to be set

Issues Resolved

List any issues this PR will resolve, e.g. Closes [...].

By 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.

Signed-off-by: Derek Ho <dxho@amazon.com>
@derek-ho
Copy link
Author

@derek-ho
Copy link
Author

@prudhvigodithi @swoehrl-mw can you folks take a look at this PR and let me know if it is sufficient? I saw a few tests that are using old versions of OpenSearch i.e. 1.x, < 2.12.0 which probably don't need updating, so I mainly fixed up the documentation. Let me know what you think, thanks!

@DarshitChanpura
Copy link
Member

@prudhvigodithi @swoehrl-mw Can we get reviews on this PR?

@DarshitChanpura
Copy link
Member

DarshitChanpura commented Jan 31, 2024

@prudhvigodithi @swoehrl-mw @idanl21 @dbason nudge. Can we get reviews on this PR?

@prudhvigodithi
Copy link
Collaborator

Hey @DarshitChanpura a recent PR has been merged #710 and the operator is waiting for a release #709 (comment). I will get back once we have stable release.
Thanks
@swoehrl-mw @bbarani

@DarshitChanpura
Copy link
Member

@prudhvigodithi Is this PR still needed, since #703 is closed?

@prudhvigodithi
Copy link
Collaborator

No @DarshitChanpura we can close the PR, thanks @derek-ho.
Adding @bbarani @peterzhuamazon

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants