Merge lp:~michael.nelson/charms/trusty/elasticsearch/merge-elasticsearch2-branch into lp:charms/trusty/elasticsearch
Proposed by
Michael Nelson
Status: | Needs review |
---|---|
Proposed branch: | lp:~michael.nelson/charms/trusty/elasticsearch/merge-elasticsearch2-branch |
Merge into: | lp:charms/trusty/elasticsearch |
Diff against target: |
182 lines (+115/-3) 7 files modified
config.yaml (+14/-0) files/nrpe/check_health (+41/-0) playbook.yaml (+25/-3) tasks/install-elasticsearch.yml (+15/-0) templates/elasticsearch.yml (+3/-0) templates/etc_default.j2 (+14/-0) templates/limits.conf (+3/-0) |
To merge this branch: | bzr merge lp:~michael.nelson/charms/trusty/elasticsearch/merge-elasticsearch2-branch |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
charmers | Pending | ||
Review via email: mp+288721@code.launchpad.net |
Commit message
Add heap_size option defaulting to half of available mem (and memlock for better performance).
Add nagios_health_color options so development deploys of 1 unit can be yellow without nagios failures.
Ensure ES is restarted on boot.
Description of the change
Since we're updating the defaults in the stable ES charm, here are the other changes we'd since added and tested with ES2. They don't seem to be ES2 specific, but I've not verified with ES1.x.
To post a comment you must log in.
Unmerged revisions
- 43. By Michael Nelson
-
Merge onlineservices elasticsearch2 branch.
Hi, thanks a lot for sending in your code!
It generally looks good, but I have small concerns with the parameters (see inline comments).