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

fix: external redis functionality #1547

Closed
wants to merge 1 commit into from
Closed

fix: external redis functionality #1547

wants to merge 1 commit into from

Conversation

iamyogeshg
Copy link
Contributor

@iamyogeshg iamyogeshg commented Oct 15, 2024

This PR extends on to the feature by #1492

We are using external redis from aws elasticache in non cluster mode.

externalRedis:
  host: master.uat-XXXXXX.amazonaws.com
  existingSecret: sentry
  existingSecretKey: redis-password
  ssl: true

redis:
  enabled: false

With above config we were facing this issue on sentry relay

2024-10-14T04:00:01.703356Z ERROR relay_server::services::processor: error processing envelope tags.project_key=834fe1715ec71e6042077d96dbde5ca4 error=failed to apply quotas error.sources=[failed to communicate with redis, failed to communicate with redis, NOAUTH: Authentication required.]

As per this issue, we have to provide username:password format and if not, we should precede the password with the colon which was unintentionally missed out for relay service. Please note we did have the colon here in the same PR.

@iamyogeshg iamyogeshg closed this by deleting the head repository Oct 15, 2024
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