Skip to content

Commit 8c2b008

Browse files
committed
fix(roles/logstash): default value of logstash__java_opts now caps jvm heap size at 8g
1 parent 2e21eec commit 8c2b008

2 files changed

Lines changed: 3 additions & 2 deletions

File tree

CHANGELOG.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -25,6 +25,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
2525

2626
### Fixed
2727

28+
* **role:logstash**: Default value of `logstash__java_opts` now caps JVM heap size at 8g.
2829
* **role:logstash**: Default value of `logstash__java_opts` now sets JVM heap size to be 60% of total memory.
2930
* **role:graylog_datanode**: Validate that `graylog_datanode__password_secret | length >= 16`
3031
* **role:graylog_server**: Validate that `graylog_server__password_secret | length >= 16`

roles/logstash/defaults/main.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -11,8 +11,8 @@ logstash__grok_patterns__host_var: []
1111
logstash__grok_patterns__role_var: []
1212
logstash__java_opts: >
1313
-Djava.io.tmpdir={{ logstash__path_data }}/tmp
14-
-Xms{{ (ansible_facts["memtotal_mb"] * 0.6) | int }}m
15-
-Xmx{{ (ansible_facts["memtotal_mb"] * 0.6) | int }}m
14+
-Xms{{ [((ansible_facts["memtotal_mb"] * 0.6) | int), 8192] | min }}m
15+
-Xmx{{ [((ansible_facts["memtotal_mb"] * 0.6) | int), 8192] | min }}m
1616
logstash__log_level: 'info'
1717
logstash__node_name: '{{ ansible_facts["nodename"] }}'
1818
logstash__path_data: '/var/lib/logstash'

0 commit comments

Comments
 (0)