search

palavatv / palava

palava browser-to-browser video
https://palava.tv
30 stars 3 forks source link

redis_address is not used for stats #9

Closed thammi closed 5 years ago

thammi commented 10 years ago

Issue by Andrwe Saturday Jan 25, 2014 at 12:35 GMT Originally opened as https://github.com/palavatv/palava-machine/issues/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 commented 5 years ago

See https://github.com/palavatv/palava-machine/pull/6

janlelis commented 5 years ago

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