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

feat: support for no sentinel setup #1987

Closed
wants to merge 2 commits into from

Conversation

jokruger
Copy link
Contributor

@jokruger jokruger commented Jun 3, 2020

In case of GCP or AWS managed redis services there will be no sentinel (HA is handled by GCP/AWS). So, need a way to setup pybossa to use redis without sentinel.

With this fix you can set REDIS_SENTINEL setting from settings_local.py to None and set REDIS_HOST and REDIS_PORT instead. In this case pybossa will use redis directly.

In case of REDIS_SENTINEL provided, sentinel will be used.

@teleyinex
Copy link
Member

@jokruger the tests are passing, so we can merge but we would need an extra step to merge this new feature into PYBOSSA:

  • Docs. Add a new sub-section with the new settings, and explain what it does. Would you mind?

If you need help, let me know it.

@teleyinex teleyinex changed the title Support for no sentinel setup feat: support for no sentinel setup Jun 6, 2020
@teleyinex
Copy link
Member

Please, reopen it when you have the docs part.

@0x2b3bfa0
Copy link

0x2b3bfa0 commented Jan 24, 2021

🔔   @teleyinex, I'm also interested in this feature and documented it through Scifabric/docs.pybossa.com#57. It would be really nice to have this merged if you deem it opprtune.

imonthercks added a commit to helpfulengineering/project-mechanical-volunteer that referenced this pull request Feb 1, 2021
@0x2b3bfa0
Copy link

Merged with #2007

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.

3 participants