Skip to content

test: live cluster resharding coverage - #13

Closed
Elijas wants to merge 2 commits into
mainfrom
fix/cluster-resharding
Closed

test: live cluster resharding coverage#13
Elijas wants to merge 2 commits into
mainfrom
fix/cluster-resharding

Conversation

@Elijas

@Elijas Elijas commented Jul 6, 2026

Copy link
Copy Markdown
Owner

Adds a live slot-migration (resharding) integration test against the real multi-node CI cluster: CLUSTER SETSLOT + MIGRATE handshake while a lease is outstanding, then lease renewal, ack, and a fresh claim through the gateway — pinning redis-py's transparent MOVED-following below client.eval. Gated on REDIS_CLUSTER_URL; wired into the real-redis-cluster CI job. Draft for CI proof only — do not merge via GitHub; the orchestrator merges locally.

Elijas added 2 commits July 7, 2026 00:51
Add tests/test_integration_cluster_resharding.py: performs a real
CLUSTER SETSLOT + MIGRATE handshake on the multi-node CI cluster while
a lease is outstanding, then drives lease renewal, ack, and a fresh
publish/claim cycle through the gateway, pinning redis-py's transparent
MOVED-following below client.eval. Gated on REDIS_CLUSTER_URL like its
siblings; migrates the slot back afterwards so cluster state stays
clean. Wire the new file into the real-redis-cluster CI job.
The cluster-coverage bullet now reflects that transparent MOVED-following
during a live slot migration is exercised in CI, narrowing the remaining
gap to partial/failed-migration redirects, multi-slot resharding storms,
and the exhausted-redirect-retries degraded path (simulation-only).
@Elijas

Elijas commented Jul 6, 2026

Copy link
Copy Markdown
Owner Author

Content merged to main locally (merge 45c55e7, on main as db83a02); closing without merge.

@Elijas Elijas closed this Jul 6, 2026
@Elijas
Elijas deleted the fix/cluster-resharding branch July 6, 2026 23:31
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