Skip to content

feat(router): make atenet timeouts configurable#179

Open
Falven (Falven) wants to merge 1 commit into
agent-substrate:mainfrom
Falven:feat/configurable-timeouts
Open

feat(router): make atenet timeouts configurable#179
Falven (Falven) wants to merge 1 commit into
agent-substrate:mainfrom
Falven:feat/configurable-timeouts

Conversation

@Falven
Copy link
Copy Markdown

Overview

This PR makes the atenet router timeout budgets configurable while preserving the current defaults.

Motivation

Some actor resume paths can exceed the current hard-coded ext_proc and route budgets. When that happens, Envoy can time out request processing before the router finishes the resume path. Operators should be able to tune these budgets per environment without recompiling the router.

Implementation

  • Add --extproc-timeout with the existing 5s default for Envoy ext_proc gRPC and message processing.
  • Add --actor-resume-timeout with the existing 15s default for actor resume calls made during request routing.
  • Add --route-timeout with the existing 10s default; 0s disables the Envoy route timeout.
  • Thread configured durations through router startup, xDS, ext_proc, and ActorResumer.
  • Add router tests for defaults, overrides, disabled route timeout, and actor resume deadline propagation.

Related to #166.

Verification

  • GOTOOLCHAIN=go1.25.0+auto GOWORK=off go test ./cmd/atenet/internal/app/router
  • Appropriate changes to documentation are included in the PR

@google-cla
Copy link
Copy Markdown

google-cla Bot commented Jun 4, 2026

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

@Falven Falven (Falven) force-pushed the feat/configurable-timeouts branch 2 times, most recently from 951c2b5 to bca32e7 Compare June 5, 2026 17:09
Add CLI configuration for Envoy ext_proc, route, and actor resume timeout budgets so deployments can tune routing latency without code changes. Thread configured durations through xDS, ext_proc, and actor resume paths and cover defaults, overrides, and disabled route timeout in focused router tests.
@Falven Falven (Falven) force-pushed the feat/configurable-timeouts branch from bca32e7 to 93c0f9f Compare June 5, 2026 17:33
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