palava-machine icon indicating copy to clipboard operation
palava-machine copied to clipboard

redis_address is not used for stats

Open Andrwe opened this issue 10 years ago • 0 comments

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

Andrwe avatar Jan 25 '14 12:01 Andrwe