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

Updated the Sidekiq readme. #129

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

Updated the Sidekiq readme. #129

wants to merge 2 commits into from

Conversation

seanmarcia
Copy link
Contributor

I ran into some issues while installing Sidekiq and I've updated the Readme in case anyone else has the same issues.

@parhamr
Copy link

parhamr commented Jan 9, 2014

@emachnic by the Chef dependency, are you referring to the depends statements in metadata.rb?

@emachnic
Copy link
Contributor

emachnic commented Jan 9, 2014

@parhamr I was referring to the README

@parhamr
Copy link

parhamr commented Jan 9, 2014

@emachnic thanks!

@radamanthus
Copy link
Contributor

Hello. Thank you for this PR. I apologise that it has languished unmerged for so long.

I'm merging this, but I will remove the section about using Redis in the DB instance in a subsequent commit. The Redis server on the DB master was there for legacy reasons, and we do not recommend using it. Redis has been removed from DB instances on our newer V5 stack.

@radamanthus
Copy link
Contributor

Unfortunately I can't merge this as is, as it has diverged significantly from the mainstream and there are now merge conflicts.

dalikim pushed a commit to parti-coop/ey-cloud-recipes that referenced this pull request Sep 18, 2016
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.

4 participants