Skip to content
This repository has been archived by the owner on Nov 7, 2018. It is now read-only.

Allow modifying elasticsearch.yml using environment variables #52

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

patricktoelle
Copy link

To add any additional configuration to elasticsearch.yml, you can use environment variables ES_CONFIG_*. As already done for elasticsearch-keystore, all variables starting with ES_CONFIG_ get evaluated and will be placed in elasticsearch.yml before starting ES.

This PR needs to get #50 to get merged first to avoid merge conflicts.

Copy link
Owner

@pires pires left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not comfortable with this PR. Also, the commit log needs to be cleaned-up.

@@ -123,6 +123,8 @@ This image can be configured by means of environment variables, that one can set
* [MEMORY_LOCK](https://www.elastic.co/guide/en/elasticsearch/reference/current/important-settings.html#bootstrap.memory_lock) - memory locking control - enable to prevent swap (default = `true`) .
* [REPO_LOCATIONS](https://www.elastic.co/guide/en/elasticsearch/reference/current/modules-snapshots.html#_shared_file_system_repository) - list of registered repository locations. For example `["/backup"]` (default = `[]`).
* [PROCESSORS](https://github.com/elastic/elasticsearch-definitive-guide/pull/679/files) - allow elasticsearch to optimize for the actual number of available cpus (must be an integer - default = 1)
* ES_CONFIG_* - add any entries to elasticsearch.yml by prodiving multiple variables. Double underscore will be replaced with dots, e.g. ES_CONFIG_S3__CLIENT__MINIO__PROTOCOL='http' ends up as s3.client.minio.protocol: http
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should be properly documented in a separate section.

* commit 'a928e6e':
  Upgrade Elasticsearch to version 6.1.3
@pires pires force-pushed the master branch 3 times, most recently from 738723d to e14ad80 Compare June 14, 2018 21:36
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants