Skip to content

Commit

Permalink
disabled auto-reload (memory leak) (see #41; reverts #30) and updated…
Browse files Browse the repository at this point in the history
… documentation
  • Loading branch information
spujadas committed May 3, 2016
1 parent b1aedff commit 94392c5
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 2 deletions.
4 changes: 3 additions & 1 deletion docs/index.md
Original file line number Diff line number Diff line change
Expand Up @@ -202,7 +202,9 @@ The following environment variables can be used to override the defaults used to

- `LS_HEAP_SIZE`: Logstash heap size (default: `"500m"`)

- `LS_OPTS`: Logstash options (default: `"--auto-reload"`)
- `LS_OPTS`: Logstash options (default: `"--auto-reload"` in images with tags `es231_l231_k450` and `es232_l232_k450`, `""` in `latest`)

**Warning** – As Logstash's auto-reload feature appears to create a resource leak (see [https://github.com/elastic/logstash/issues/5235](https://github.com/elastic/logstash/issues/5235)), the default `--auto-reload` option has been removed in the `latest` image at the time of writing (see [https://github.com/spujadas/elk-docker/issues/41](https://github.com/spujadas/elk-docker/issues/41)). For users of images with tags `es231_l231_k450` and `es232_l232_k450`, it is strongly recommended to override Logstash's options to disable the auto-reload feature by setting the `LS_OPTS` environment to `--no-auto-reload`. Future versions of the image may re-enable the auto-reload feature once the memory leak issue is fixed.

As an illustration, the following command starts the stack, running Elasticsarch with a 2GB heap size, Logstash with a 1GB heap size and Logstash's configuration auto-reload disabled:

Expand Down
2 changes: 1 addition & 1 deletion logstash-init
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,7 @@ LS_LOG_FILE="${LS_LOG_DIR}/$name.log"
LS_CONF_DIR=/etc/logstash/conf.d
LS_OPEN_FILES=16384
LS_NICE=19
LS_OPTS="--auto-reload"
LS_OPTS=

[ -r /etc/default/$name ] && . /etc/default/$name
[ -r /etc/sysconfig/$name ] && . /etc/sysconfig/$name
Expand Down

0 comments on commit 94392c5

Please sign in to comment.