Skip to content

Remove the default dispatch stream deadline - #355

Open
Marenz wants to merge 1 commit into
frequenz-floss:v1.x.xfrom
Marenz:fix/no-default-stream-deadline
Open

Marenz wants to merge 1 commit into
frequenz-floss:v1.x.xfrom
Marenz:fix/no-default-stream-deadline

Conversation

@Marenz

@Marenz Marenz commented Nov 11, 2026

Copy link
Copy Markdown
Contributor

Dispatch streams currently reconnect every five minutes even while healthy because the client applies a default deadline.

Default stream_timeout to None so gRPC keepalive detects dead connections. Callers can still pass a finite timeout; unary deadlines are unchanged.

Healthy dispatch streams should not reconnect on a timer. Let gRPC keepalive
detect dead connections unless the caller requests a finite stream timeout.

Signed-off-by: Mathias L. Baumann <mathias.baumann@frequenz.com>
@Marenz
Marenz requested review from a team as code owners November 11, 2026 09:47
@Marenz
Marenz requested review from ela-kotulska-frequenz and florian-wagner-frequenz and removed request for a team November 11, 2026 09:47
@github-actions github-actions Bot added part:docs Affects the documentation part:tests Affects the unit, integration and performance (benchmarks) tests part:dispatcher labels Nov 11, 2026
@Marenz
Marenz enabled auto-merge November 11, 2026 09:49

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

part:dispatcher part:docs Affects the documentation part:tests Affects the unit, integration and performance (benchmarks) tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants