-
Notifications
You must be signed in to change notification settings - Fork 466
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 docker.md #6519
Update docker.md #6519
Conversation
Adding OPENSEARCH_INITIAL_ADMIN_PASSWORD variable to OpenSearch node for version 2.12.0. Signed-off-by: Pawel Wlodarczyk <pawel.wlodarczyk@eliatra.com>
@derek-ho or @peterzhuamazon - Can you please approve this from a technical standpoint? Thanks! |
@@ -170,6 +170,7 @@ services: | |||
- cluster.initial_cluster_manager_nodes=opensearch-node1,opensearch-node2 # Nodes eligible to serve as cluster manager | |||
- bootstrap.memory_lock=true # Disable JVM heap memory swapping | |||
- "OPENSEARCH_JAVA_OPTS=-Xms512m -Xmx512m" # Set min and max JVM heap sizes to at least 50% of system RAM | |||
- OPENSEARCH_INITIAL_ADMIN_PASSWORD=${OPENSEARCH_INITIAL_ADMIN_PASSWORD} # Sets the demo admin user password when using demo configuration, required for OpenSearch 2.12 and higher |
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.
This looks good to me, but I think we may want to call out whether this is set here, or whether this env variable is present from outside or an .env file. Tagging @DarshitChanpura @peterzhuamazon to leave their comments here as well.
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.
we should add a note on line 157 stating the requirement to either define the variable in a .env
and exporting it in the same terminal session where docker-compose is run from.
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.
Also, I have a companion PR to update the download steps for docker: opensearch-project/project-website#2526
Signed-off-by: Heather Halter <HDHALTER@AMAZON.COM>
Signed-off-by: Heather Halter <HDHALTER@AMAZON.COM>
Co-authored-by: Naarcha-AWS <97990722+Naarcha-AWS@users.noreply.github.com> Signed-off-by: Heather Halter <HDHALTER@AMAZON.COM>
Signed-off-by: Heather Halter <HDHALTER@AMAZON.COM>
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.
One last change then LGTM.
Co-authored-by: Naarcha-AWS <97990722+Naarcha-AWS@users.noreply.github.com> Signed-off-by: Heather Halter <HDHALTER@AMAZON.COM>
* Update docker.md Adding OPENSEARCH_INITIAL_ADMIN_PASSWORD variable to OpenSearch node for version 2.12.0. Signed-off-by: Pawel Wlodarczyk <pawel.wlodarczyk@eliatra.com> * Added note about 2.12 Signed-off-by: Heather Halter <HDHALTER@AMAZON.COM> * Update _install-and-configure/install-opensearch/docker.md Signed-off-by: Heather Halter <HDHALTER@AMAZON.COM> * Update _install-and-configure/install-opensearch/docker.md Co-authored-by: Naarcha-AWS <97990722+Naarcha-AWS@users.noreply.github.com> Signed-off-by: Heather Halter <HDHALTER@AMAZON.COM> * Update docker.md Signed-off-by: Heather Halter <HDHALTER@AMAZON.COM> * Update _install-and-configure/install-opensearch/docker.md Co-authored-by: Naarcha-AWS <97990722+Naarcha-AWS@users.noreply.github.com> Signed-off-by: Heather Halter <HDHALTER@AMAZON.COM> --------- Signed-off-by: Pawel Wlodarczyk <pawel.wlodarczyk@eliatra.com> Signed-off-by: Heather Halter <HDHALTER@AMAZON.COM> Co-authored-by: Heather Halter <HDHALTER@AMAZON.COM> Co-authored-by: Naarcha-AWS <97990722+Naarcha-AWS@users.noreply.github.com> (cherry picked from commit 083496a) Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
* Update docker.md Adding OPENSEARCH_INITIAL_ADMIN_PASSWORD variable to OpenSearch node for version 2.12.0. * Added note about 2.12 * Update _install-and-configure/install-opensearch/docker.md * Update _install-and-configure/install-opensearch/docker.md * Update docker.md * Update _install-and-configure/install-opensearch/docker.md --------- (cherry picked from commit 083496a) Signed-off-by: Pawel Wlodarczyk <pawel.wlodarczyk@eliatra.com> Signed-off-by: Heather Halter <HDHALTER@AMAZON.COM> Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> Co-authored-by: Heather Halter <HDHALTER@AMAZON.COM> Co-authored-by: Naarcha-AWS <97990722+Naarcha-AWS@users.noreply.github.com>
* Update docker.md Adding OPENSEARCH_INITIAL_ADMIN_PASSWORD variable to OpenSearch node for version 2.12.0. Signed-off-by: Pawel Wlodarczyk <pawel.wlodarczyk@eliatra.com> * Added note about 2.12 Signed-off-by: Heather Halter <HDHALTER@AMAZON.COM> * Update _install-and-configure/install-opensearch/docker.md Signed-off-by: Heather Halter <HDHALTER@AMAZON.COM> * Update _install-and-configure/install-opensearch/docker.md Co-authored-by: Naarcha-AWS <97990722+Naarcha-AWS@users.noreply.github.com> Signed-off-by: Heather Halter <HDHALTER@AMAZON.COM> * Update docker.md Signed-off-by: Heather Halter <HDHALTER@AMAZON.COM> * Update _install-and-configure/install-opensearch/docker.md Co-authored-by: Naarcha-AWS <97990722+Naarcha-AWS@users.noreply.github.com> Signed-off-by: Heather Halter <HDHALTER@AMAZON.COM> --------- Signed-off-by: Pawel Wlodarczyk <pawel.wlodarczyk@eliatra.com> Signed-off-by: Heather Halter <HDHALTER@AMAZON.COM> Co-authored-by: Heather Halter <HDHALTER@AMAZON.COM> Co-authored-by: Naarcha-AWS <97990722+Naarcha-AWS@users.noreply.github.com>
Adding OPENSEARCH_INITIAL_ADMIN_PASSWORD variable to OpenSearch node for version 2.12.0.
Description
Adding OPENSEARCH_INITIAL_ADMIN_PASSWORD variable to OpenSearch nodes in example docker-compose.yml file for version 2.12.0.
Checklist
For more information on following Developer Certificate of Origin and signing off your commits, please check here.