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

redis_address is not used for stats #9

Closed
thammi opened this issue Nov 23, 2014 · 2 comments
Closed

redis_address is not used for stats #9

thammi opened this issue Nov 23, 2014 · 2 comments

Comments

@thammi
Copy link
Member

thammi commented Nov 23, 2014

Issue by Andrwe
Saturday Jan 25, 2014 at 12:35 GMT
Originally opened as palavatv/palava-machine#4


Hi,

in jobs/export_stats_job.rb you define initialize() with two parameters "redis_address" and "mongo_address" which can be given via command line.

But you don't use redis_address to start the redis-connection.
Instead you use hardcoded localhost:6379.

Thus you can't change the redis-server configuration withour losing the stats-mechanism.

The call should be changed to something like:
EM::Hiredis.connect "redis://#{@redis_address}/#{@redis_db}" (used in manager.rb)

Regards,
Andrwe

@janlelis
Copy link
Member

See palavatv/palava-machine#6

@janlelis janlelis reopened this Mar 17, 2019
@janlelis
Copy link
Member

machine is not part of default palava stack anymore. please use machine repo for issues

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

No branches or pull requests

2 participants