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

Redis client with AWS Elasticache #32159

Closed
OmerShemesh opened this issue Mar 27, 2023 · 5 comments · Fixed by #43566
Closed

Redis client with AWS Elasticache #32159

OmerShemesh opened this issue Mar 27, 2023 · 5 comments · Fixed by #43566
Assignees
Labels
Milestone

Comments

@OmerShemesh
Copy link

Describe the bug

I've created a redis cluster using AWS elasticache (with no cluster mode).
The cluster contains a master and 2 replicas.

I'm building a Lambda using the quarkus-amazon-lambda extension as well as the quarkus-redis-client extension.
I've configured my client to use this type: quarkus.redis.client-type=replication.

I've deployed my lambda and configured it to be in the same VPC as my Elasticache Redis cluster.

When invoking the lambda, I'm getting this error:

2023-03-27 13:50:25,852 WARN  [io.ver.red.cli.imp.RedisReplicationClient] (vert.x-eventloop-thread-1) Skipping failed node: 10.21.0.58: io.netty.channel.ConnectTimeoutException: connection timed out: /x.x.x.x:6379

The ip in the error(x.x.x.x) is outside of my VPC.

Turns out that Vertx RedisReplicationClient is using the info replication command that returns:

connected_slaves:2
slave0:ip=x.x.x.x.,port=6379,state=online,offset=5947958,lag=1
slave1:ip=y.y.y.y,port=6379,state=online,offset=5947958,lag=1

Where the ips from the command are indeed the ips from the error above.

This is clearly a problem with the Vertx client.
I've tried to deploy a simple lambda using nodejs and the redis dependency and it worked as expected.

Expected behavior

A successful connection from the lambda to Elasticache.

Actual behavior

No response

How to Reproduce?

No response

Output of uname -a or ver

No response

Output of java -version

No response

GraalVM version (if different from Java)

No response

Quarkus version or git rev

No response

Build tool (ie. output of mvnw --version or gradlew --version)

No response

Additional information

No response

@OmerShemesh OmerShemesh added the kind/bug Something isn't working label Mar 27, 2023
@quarkus-bot
Copy link

quarkus-bot bot commented Mar 27, 2023

/cc @cescoffier (redis), @gsmet (redis), @gwenneg (cache), @machi1990 (redis)

@primozh
Copy link
Contributor

primozh commented Mar 30, 2023

Possibly related to this one: vert-x3/vertx-redis-client#343

@cescoffier cescoffier assigned cescoffier and unassigned cescoffier Jul 3, 2023
@cescoffier
Copy link
Member

Closing as vert-x3/vertx-redis-client#343 has been fixed and integrated in Quarkus.

Please re-open it with a reproducer if you are still having the issue.

@cescoffier cescoffier closed this as not planned Won't fix, can't repro, duplicate, stale Jul 25, 2024
@cescoffier cescoffier added the triage/out-of-date This issue/PR is no longer valid or relevant label Jul 25, 2024
@Ladicek Ladicek self-assigned this Jul 25, 2024
@Ladicek Ladicek reopened this Jul 25, 2024
@Ladicek
Copy link
Contributor

Ladicek commented Jul 25, 2024

Let me reopen this, because vert-x3/vertx-redis-client#343 has not been fixed in a Vert.x release yet. And when we do integrate it, we'll have to adjust the configuration model in Quarkus too. So the remaining work is small, but non-zero still.

@Ladicek Ladicek removed the triage/out-of-date This issue/PR is no longer valid or relevant label Jul 25, 2024
@cescoffier
Copy link
Member

My bad! I was thinking it was integrated

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

Successfully merging a pull request may close this issue.

4 participants