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

Support for Redis Sentinel #5475

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Ithanil
Copy link
Contributor

@Ithanil Ithanil commented Oct 20, 2023

This PR enables the use of Redis Sentinel Clusters with GL3. We are using this feature in production, but it was not developed with the goal to be merged upstream. However, I'd like to provide our development to the public for anyone in need of this and would be happy to help getting this ready for merging if there is interest.

This is the beginning of a series of similar PR submissions (Redis Sentinel, SAML integration, LDAP integration).

Details / How to use:

  • See sample.env for some documention on configuration
  • Provide Redis Sentinel hostnames & ports in the variable REDIS_SENTINEL
  • If REDIS_SENTINEL is set, REDIS_URL will be redis://mymaster per default

@ffdixon
Copy link
Member

ffdixon commented Oct 21, 2023

Thank you for sharing this @Ithanil.

@Ithanil Ithanil force-pushed the redis_sentinel branch 3 times, most recently from be71b16 to b5bd59b Compare November 3, 2023 15:59
Copy link

sonarqubecloud bot commented Nov 3, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

Copy link

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

Copy link

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants