Open batuhan opened 2 years ago
APM works, so thats good.
disabling signal/websockets feature helps but not does solve this issue completely since we are not solving the problem but delaying it. getPlatformData
stuff should just return values from the env and we shouldn't depend on redis at all.
this might have implications for the tests we are running but that's also a design flaw.
We get the following error, both on local and production when using the production redis installation. We might be hitting the connection limit on the cloud (digitalocean for now but aws has similar restrictions).
Code at
./services/core/src/redis.js
. It does use a singleton.This is critical since the redis connection is essential for workers, signal & multi tenantness of our platform thus a failed redis connection should (and currently does) result in an exited process
We might think about stripping in favour of something like SQS for the workers, Firebase etc. for Signal and env. variables for the multi tenantness but this would add time to our time to deploy.