Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Sidekiq config per host #81

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

Conversation

klacointe
Copy link

Allow to configure sidekiq config file per host.

Follow the same logic as processes configuration.

@klacointe klacointe force-pushed the sidekiq_config_per_host branch from 5ca91e0 to 02fbc09 Compare May 7, 2015 10:02
@lustercr
Copy link

I was testing these changes.

gem 'capistrano-sidekiq', github: 'klacointe/capistrano-sidekiq', :branch => "sidekiq_config_per_host"

and this is how my deploy.rb file looks like

set :sidekiq_role, [:sidekiq_readonly, :sidekiq_general]
set :sidekiq_general_config, "#{current_path}/config/sidekiqXXX.yml"
set :sidekiq_readonly_config, "#{current_path}/config/sidekiq_YYYY.yml"
server 'serverXXXXX', roles: [:sidekiq_general]
server 'serverYYYYY', roles: [:sidekiq_readonly]

worked fine for me.

@seuros are we planning to test/merge this any soon ?

@okoriko
Copy link
Contributor

okoriko commented Aug 14, 2015

👍

@seuros any hope of this getting merged?
I don't mind writing a patch to support these changes on the monit side.

@seuros
Copy link
Owner

seuros commented Aug 19, 2015

Sorry for delay. I will implement this feature this weekend max.

@w1mvy w1mvy mentioned this pull request Nov 12, 2015
@lustercr
Copy link

Hi @seuros, noticed that you added support to "Different number of processes per host" do this also mean that we can specify a different config file ? my use case is that i have different machines on which they process specific job times. some of my machines only process mails others exports, etc

thanks for your answer.

@brentdodell
Copy link

Sorry for delay. I will implement this feature this weekend max.

Hi @seuros. I see that this PR wasn't merged. Just wondering if the same functionality was integrated elsewhere.

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

Successfully merging this pull request may close these issues.

5 participants