TranslatorSRI / NodeNormalization

Service that produces Translator compliant nodes given a curie
MIT License
9 stars 6 forks source link

Update Redis backend #196

Closed gaurav closed 1 month ago

gaurav commented 1 year ago

We've been having some issues with the Redis servers behind NodeNode recently. In order to debug this issues, it might help to fix some issues in the Kubernetes file we use to set this up at https://github.com/helxplatform/translator-devops/tree/develop/helm/redis-r3-external. Some possible improvements:

Note that we don't need to worry about this at all for ITRB, because they run and manage their own Redis cluster instance. At some point it might make sense for us to do this as well, but the Redis-on-Kubernetes setup we currently use seems to work fine as long as it doesn't have random issues.

gaurav commented 1 year ago

the current setup is hardcoded to use redis-12.7.4.tgz

Not true -- it uses the helm dependency system, so you can use https://helm.sh/docs/helm/helm_dependency_update/ to upgrade it.

gaurav commented 9 months ago

We have upgraded Redis in RENCI Dev. I'll keep this open until we've shown that the code continues to work on ITRB.

gaurav commented 1 month ago

This seems to be generally working everywhere. We do need to upgrade Redis to Redis 7 at some point, which we should coordinate with upgrading the databases on ITRB, but that's not particularly urgent. Closing.