-
Notifications
You must be signed in to change notification settings - Fork 924
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
[v2.12.0] Ensure CI/documentation reflect changes to default admin credentials #5636
Comments
Seems like the only references are for BWC tests: https://github.com/search?q=repo%3Aopensearch-project%2FOpenSearch-Dashboards%20admin%3Aadmin&type=code, latest of which is 2.9.0. I think that makes this repo compliant/I do not see any CI failures on main/2.x either. Can Maintainers confirm if this is the only workflow which relies on admin:admin default creds in this repo? Thanks! |
@kavilla are we good to close this issue? |
@wbeckler @kavilla @ashwin-pc Can you please provide your updates? |
@kavilla would you be willing to take a look at https://github.com/search?q=repo%3Aopensearch-project%2FOpenSearch-Dashboards++loginPage.enterPassword%28%27admin%27%29%3B&type=code |
@kavilla I found this:
|
local development. developers can pull down this repo, clone the security dashboards plugin. Then from OSD call If you just want to scale up an OpenSearch cluster with the certs install you can call |
We should mention that a strong password is required to start the cluster with demo security configuration. Otherwise, when someone attempts to run |
I think we do. Do you think this will suffice @DarshitChanpura? |
Background
Previously, when installing the security plugin demo configuration, the cluster was spun up with the default admin credentials,
admin:admin
. A change was made inmain
and backported to2.x
for the2.12.0
release, which now requires an initial admin password to be passed in via the environment variableOPENSEARCH_INITIAL_ADMIN_PASSWORD
. This will break some CI/testing that relies on OpenSearch to come up without setting this environment variable. This tracking issue is to ensure compliance with the new changes.Coming from: opensearch-project/security#3624
Acceptance Criteria
admin:admin
are removedmain
and2.x
branchesThe text was updated successfully, but these errors were encountered: