Skip to content
This repository has been archived by the owner on Aug 1, 2022. It is now read-only.

init.d script and configuration for running under Puma #49

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

init.d script and configuration for running under Puma #49

wants to merge 6 commits into from

Conversation

jimothyGator
Copy link

This adds the following:

  • init.d script
  • Rake task for starting/stopping Sidekiq
  • Puma configuration

This is based on the instructions for running Gitlab and has been tested on Ubuntu.

@joelmoss
Copy link
Owner

I'm not so sure hat we want to require using Puma here. And this will also conflict with the use of Thin in the development bundle group. Lets use one or the other.

Can you also rebase these commits into one please? thx

@jimothyGator
Copy link
Author

Okay, I can switch it over to run with Thin, and rebase to a single commit.

I picked Puma primarily because I cribbed this setup from Gitlab, which switched from Unicorn to Puma to reduce memory footprint (and the reduction is significant), but honestly, I hadn't compared it to Thin.

@joelmoss
Copy link
Owner

I'm happy to use either, but lets stick with one.

Joel Moss

Develop with Style at http://DevelopWithStyle.com

Call me +44 7791 503309
http://twitter.com/joelmoss
AIM: joelkmoss
Skype: joelmoss.info (http://joelmoss.info)

On Tuesday, 28 May 2013 at 04:38, Jim Cushing wrote:

Okay, I can switch it over to run with Thin, and rebase to a single commit.
I picked Puma primarily because I cribbed this setup from Gitlab, which switched from Unicorn to Puma to reduce memory footprint (and the reduction is significant), but honestly, I hadn't compared it to Thin.


Reply to this email directly or view it on GitHub (#49 (comment)).

@jimothyGator
Copy link
Author

Sorry for the wait; it's been a busy week. I'll get this cleaned up this weekend.

@jimothyGator
Copy link
Author

I've written more detail installation instructions. I am going to do some clean up, re-test these with a fresh Debian install, and then I will re-base as a single commit.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants