Juju Charms Collection
logstash package

Merge lp://staging/~cprov/charms/trusty/logstash/heap_size into lp://staging/~tanuki/charms/trusty/logstash/trunk

  1. Trusty Tahr (14.04)
  2. heap_size
  3. Merge into trunk
Proposed by Celso Providelo
Status: Superseded
Proposed branch: lp://staging/~cprov/charms/trusty/logstash/heap_size
Merge into: lp://staging/~tanuki/charms/trusty/logstash/trunk
Diff against target: 736 lines (+451/-85)
13 files modified
README.md (+0/-21)
config.yaml (+34/-4)
files/nrpe/check_e2e_output.py (+59/-0)
files/nrpe/check_lumberjack_e2e.py (+265/-0)
files/upstart/logstash-indexer.conf (+2/-1)
hooks/client-relation-changed (+13/-29)
hooks/config-changed (+27/-10)
hooks/nrpe-external-master-relation-changed (+43/-2)
templates/e2e-cron.tmpl (+4/-0)
templates/input-file-syslog.conf (+1/-1)
templates/input-python-logging.conf (+0/-14)
templates/input-redis.conf (+1/-1)
templates/output-elasticsearch.conf (+2/-2)
To merge this branch: bzr merge lp://staging/~cprov/charms/trusty/logstash/heap_size
Reviewer Review Type Date Requested Status
Michael Nelson (community) Approve
Guillermo Gonzalez Needs Information
Review via email: mp+311665@code.staging.launchpad.net

This proposal has been superseded by a proposal from 2016-11-23.

Commit message

Allowing changing the logstash maximum heap size as a configuration.

Description of the change

Allowing changing the logstash maximum heap size as a configuration.

Defaults to 1000m instead of the application 500m default value and allow adjustments.

To post a comment you must log in.
lp://staging/~cprov/charms/trusty/logstash/heap_size updated
59. By Celso Providelo

Adding heap_size config option.

Revision history for this message
Guillermo Gonzalez (verterok) wrote :

Celso,

If this is to apply it to the OLS ELK, it's using a a different charm:

lp:~canonical-is-sa/charms/trusty/logstash/logstash2;revno=61,overwrite=true

the spec in use (for both staging and production) is: is/mojo-is-logging-kibana/ @ lp:canonical-mojo-specs.

review: Needs Information
Revision history for this message
Michael Nelson (michael.nelson) wrote :

LGTM. Note that in the ES charm, we're explicitly giving unlimited memlock. See the task at:

http://bazaar.launchpad.net/~onlineservices-charmers/charms/trusty/elasticsearch/elasticsearch2/view/head:/tasks/install-elasticsearch.yml#L49

which installs this template:

http://bazaar.launchpad.net/~onlineservices-charmers/charms/trusty/elasticsearch/elasticsearch2/view/head:/templates/limits.conf

review: Approve
lp://staging/~cprov/charms/trusty/logstash/heap_size updated
60. By Celso Providelo

merge right branch (lp:~canonical-is-sa/charms/trusty/logstash/logstash2).

61. By Celso Providelo

Special (no-) limits configuration for logstash, so it can eat as much memory it is configured too.

Unmerged revisions

61. By Celso Providelo

Special (no-) limits configuration for logstash, so it can eat as much memory it is configured too.

60. By Celso Providelo

merge right branch (lp:~canonical-is-sa/charms/trusty/logstash/logstash2).

59. By Celso Providelo

Adding heap_size config option.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches

to all changes: