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

WIP redis sentinel #279

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from
Draft

WIP redis sentinel #279

wants to merge 3 commits into from

Conversation

stgrace
Copy link
Contributor

@stgrace stgrace commented Jul 23, 2021

Description of feature

Support for redis sentinel

Additional information

So far added redis_host has a list of tuples, just like in arq.
Attempted to use a Union of tuples and str, but when I did it always parsed the conf through the environment as a string.

TODO

  • Change stackl-redis entrypoint for sentinel (sentinel.conf must be writeable, so the config mounted from a cm cannot be used directly. Best to probably cp initial conf to empty-dir volume and use that conf, so stackl-redis can write to it. See how bitnami does it)
  • Add connection to multiple redis for stackl-core (datastore and queue).
  • Fix Union issue for redis_hosts config
  • Run tests to see if data is not lost when failover happens

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

Successfully merging this pull request may close these issues.

1 participant