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

Proposed by James Page
Status: Merged
Merged at revision: 56
Proposed branch: lp:~james-page/charms/precise/rabbitmq-server/ulimit-increase
Merge into: lp: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:~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.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
=== modified file 'hooks/rabbitmq_server_relations.py'
--- hooks/rabbitmq_server_relations.py 2014-05-02 14:18:00 +0000
+++ hooks/rabbitmq_server_relations.py 2014-05-23 08:13:18 +0000
@@ -515,7 +515,8 @@
515 # Add archive source if provided515 # Add archive source if provided
516 add_source(config('source'), config('key'))516 add_source(config('source'), config('key'))
517 apt_update(fatal=True)517 apt_update(fatal=True)
518518 # Copy in defaults file for updated ulimits
519 shutil.copyfile('templates/rabbitmq-server', '/etc/default/rabbitmq-server')
519 # Install packages to ensure any changes to source520 # Install packages to ensure any changes to source
520 # result in an upgrade if applicable.521 # result in an upgrade if applicable.
521 apt_install(rabbit.PACKAGES, fatal=True)522 apt_install(rabbit.PACKAGES, fatal=True)
522523
=== added file 'templates/rabbitmq-server'
--- templates/rabbitmq-server 1970-01-01 00:00:00 +0000
+++ templates/rabbitmq-server 2014-05-23 08:13:18 +0000
@@ -0,0 +1,3 @@
1# Generated by juju
2# bump ulimit so rabbit can support lots of connections
3ulimit -n 65536

Subscribers

People subscribed via source and target branches