Skip to content

fix: harden resp-compat client setup for Eloq e2e#427

Open
starrysky9959 wants to merge 5 commits into
mainfrom
fix/resp-compat-tls
Open

fix: harden resp-compat client setup for Eloq e2e#427
starrysky9959 wants to merge 5 commits into
mainfrom
fix/resp-compat-tls

Conversation

@starrysky9959

Copy link
Copy Markdown
Collaborator

Summary

  • add an Eloq-specific resp-compat wrapper that forces RESP2 for redis-py clients
  • preserve local e2e TLS compatibility for resp-compat when --ssl is enabled
  • discover the current primary from CLUSTER SLOTS before running resp-compat so tests do not target the standby node

Validation

  • rebuilt the resp-compat docker image locally
  • verified standalone and cluster clients can both connect with redis 8.0.0 when protocol=2 is forced
  • verified the resp-compat script now progresses into real test execution instead of failing during client initialization

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