Skip to content

fix(zookeeper): tolerate transient connection loss - #10437

Merged
ReubenBond merged 1 commit into
dotnet:mainfrom
ReubenBond:reubenbond-fix-zookeeper-update-flake
Aug 11, 2026
Merged

ReubenBond merged 1 commit into
dotnet:mainfrom
ReubenBond:reubenbond-fix-zookeeper-update-flake

Conversation

@ReubenBond

@ReubenBond ReubenBond commented Aug 11, 2026

Copy link
Copy Markdown
Member

The ZooKeeper provider configured a 2-second session timeout, which also bounded ZooKeeper.Using retries after a transient ConnectionLossException. The membership update test performs enough operations to intermittently exhaust that short window in CI.

Increase the session timeout to 10 seconds and name it according to the ZooKeeper client API semantics. This gives membership reads and writes time to reconnect without changing transaction behavior.

Fixes #10435

Microsoft Reviewers: Open in CodeFlow

Increase the ZooKeeper session retry window so brief disconnects do not fail membership operations.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@ReubenBond
ReubenBond merged commit 5687dca into dotnet:main Aug 11, 2026
72 checks passed
@ReubenBond
ReubenBond deleted the reubenbond-fix-zookeeper-update-flake branch August 11, 2026 02:19
This was referenced Aug 28, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Flaky test: MembershipTable_ZooKeeper_UpdateRow loses its ZooKeeper connection

1 participant