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

Proposed by James Page
Status: Merged
Merged at revision: 56
Proposed branch: lp://qastaging/~james-page/charms/precise/rabbitmq-server/ulimit-increase
Merge into: lp://qastaging/charms/rabbitmq-server
Diff against target: 21 lines (+5/-1)
2 files modified
hooks/rabbitmq_server_relations.py (+2/-1)
templates/rabbitmq-server (+3/-0)
To merge this branch: bzr merge lp://qastaging/~james-page/charms/precise/rabbitmq-server/ulimit-increase
Reviewer Review Type Date Requested Status
Jorge Niedbalski (community) Needs Information
Charles Butler (community) Approve
Review via email: mp+220434@code.qastaging.launchpad.net

Description of the change

Increase default ulimit to something more scalable.

This can be pushed to the trusty charm branch as well.

To post a comment you must log in.
55. By James Page

Tune heartbeat

56. By James Page

Backout heartbeat changes

Revision history for this message
Charles Butler (lazypower) wrote :

+1 LGTM

Thanks for the submission!

review: Approve
Revision history for this message
Charles Butler (lazypower) wrote :

Pushed to both precise/trusty.

Revision history for this message
Jorge Niedbalski (niedbalski) wrote :

Hello James,

Please review https://code.launchpad.net/~niedbalski/charms/precise/rabbitmq-server/ulimit-increase/+merge/222868

I think using /etc/security/limits.d/ would be a better option for this change.

Let me know your impression.

review: Needs Information

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