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

Error connecting to redis #2779

Closed
ViditChitkara opened this issue Jun 7, 2018 · 3 comments
Closed

Error connecting to redis #2779

ViditChitkara opened this issue Jun 7, 2018 · 3 comments

Comments

@ViditChitkara
Copy link
Member

ViditChitkara commented Jun 7, 2018

By default redis is trying to connect to localhost (127.0.0.1)

Started GET "/users/test_digest_email" for 172.56.22.214 at 2018-06-06 22:12:08 +0000
Processing by UsersController#test_digest_email as HTML
Completed 500 Internal Server Error in 7ms (ActiveRecord: 1.2ms)
  
Redis::CannotConnectError (Error connecting to Redis on 127.0.0.1:6379 (Errno::ECONNREFUSED)):
  app/controllers/users_controller.rb:301:in `test_digest_email`

Discussed at:- #2711

@ViditChitkara
Copy link
Member Author

sidekiq:
build: .
command: bundle exec sidekiq
depends_on:
- db
- redis
volumes:
- .:/app
environment:
- JOB_WORKER_URL=redis://redis:6379/0

@icarito there is an environment var named JOB_WORKER_URL in sidekiq container, won't it work? Should I add it in web container?? @jywarren any recommendations, on this? I am a bit in-experienced with docker-compose. Any help is appreciated!!

@icarito
Copy link
Member

icarito commented Jul 4, 2018

@jywarren I think this was working in unstable but we still need to put it in stable and production. I'll tweak the docker-compose-production.yml now to get this running.

@icarito icarito closed this as completed Jul 4, 2018
@ViditChitkara
Copy link
Member Author

Hi @icarito, I had made the changes for production and stable in #2772, however we saw a circular dependency error, so I made some changes in #2978. Can you have a look at both??
Thanks!

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 a pull request may close this issue.

2 participants