-
Notifications
You must be signed in to change notification settings - Fork 121
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
Pass in initialAdminPassword #669
Changes from 6 commits
eebf1ae
5cd431d
40393a1
667d3b6
9d987bf
d221bc7
47b573e
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -13,6 +13,7 @@ services: | |
- discovery.type=single-node | ||
- bootstrap.memory_lock=true | ||
- SECURE_INTEGRATION=${SECURE_INTEGRATION:-false} | ||
- OPENSEARCH_INITIAL_ADMIN_PASSWORD=myStrongPassword123! | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. @DarshitChanpura with OS 2.12 and Up, would this be enough to set the admin password? Does the plugin just grab what stored in There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I also saw this in the security plugin repo: https://github.com/opensearch-project/security/pull/3843/files There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Yes, that is correct. The env variable will be picked up when running demo configuration.
There is no concept of custom admin passwords for versions prior to 2.12. This will be introduced in 2.12 and will be required henceforth when using security plugin's demo configuration. Please note that variable is |
||
ports: | ||
- '9200:9200' | ||
user: opensearch | ||
|
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.
Should this change be version based?
admin
for <2.12 andmyStrongPassword123!
for >=2.12.