Juju Charms Collection
rabbitmq-server package

Merge lp://qastaging/~niedbalski/charms/precise/rabbitmq-server/ulimit-increase into lp://qastaging/charms/rabbitmq-server

Proposed by Jorge Niedbalski on 2014-06-11

Status:	Merged
Merged at revision:	57
Proposed branch:	lp://qastaging/~niedbalski/charms/precise/rabbitmq-server/ulimit-increase
Merge into:	lp://qastaging/charms/rabbitmq-server
Diff against target:	31 lines (+7/-4) 3 files modified hooks/rabbitmq_server_relations.py (+2/-1) templates/rabbitmq-server (+0/-3) templates/rabbitmq.limits.conf (+5/-0)
To merge this branch:	bzr merge lp://qastaging/~niedbalski/charms/precise/rabbitmq-server/ulimit-increase
Related bugs:	Link a bug report

Reviewer	Review Type	Date Requested	Status
Charles Butler (community)		2014-06-11	Approve on 2014-06-11
Review via email: mp+222868@code.qastaging.launchpad.net

Description of the change

- Alternative proposal for LP: #220434

* Increase default ulimit to something more scalable.
* This can be pushed to the trusty charm branch as well.

Revision history for this message

Charles Butler (lazypower) wrote on 2014-06-11:

+1 LGTM - i like this better in security/limits vs in defaults. TY for this merge.

review: Approve

Revision history for this message

James Page (james-page) wrote on 2014-06-12:

Does this apply correctly on a reboot? I've seen issues in the past where limits configuration is not applied correctly prior to services starting up.