Skip to content

charts/substrate: expose router timeouts with 60s defaults - #132

Merged
EItanya merged 1 commit into
mainfrom
shashankram/kagent-timeout
Nov 5, 2026
Merged

EItanya merged 1 commit into
mainfrom
shashankram/kagent-timeout

Conversation

@shashankram

@shashankram shashankram commented Nov 5, 2026 •

Copy link
Copy Markdown

Slow actor restores can exceed the router's parking budget or per-call API timeout, causing retries to restart the restore. Related to #114; extends #116.

  • Default atenetRouter.requestParking.budget and atenetRouter.requestTimeout to 60s across all four ingress routes.
  • Document how the limits interact and keep a 5s fixture for parking E2E tests.
  • Extend the existing Helm tests for defaults, independent overrides, and the E2E fixture.

AI use: Codex implemented the change, ran checks, and performed a separate agent review.

  • Tests pass
  • Docs updated

/kind fix

Router parking and API request timeouts now default to 60s and are configurable through Helm.

- Allow slow actor restores more time before the router cancels them.
- Configure parking and API request limits across all ingress routes.
- Keep a five-second parking budget for budget-exhaustion tests.

/kind fix

```release-note
Router parking and API request timeouts now default to 60s and are
configurable through Helm.
```

Signed-off-by: Shashank Ram <shashank.ram@solo.io>
@shashankram
shashankram force-pushed the shashankram/kagent-timeout branch from ade2f9c to 3a77459 Compare November 5, 2026 15:56

@danielle-mustillo danielle-mustillo left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm but I dunno if someone else wants to look

@EItanya
EItanya merged commit 4d8bf1c into main Nov 5, 2026
6 checks passed
@shashankram
shashankram deleted the shashankram/kagent-timeout branch November 5, 2026 16:42
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.

3 participants