Skip to content

[backport] charts/substrate: expose router timeouts with 60s defaults - #134

Merged
shashankram merged 1 commit into
release-1.0from
shashankram/backport-router-timeouts-1.0
Nov 5, 2026
Merged

shashankram merged 1 commit into
release-1.0from
shashankram/backport-router-timeouts-1.0

Conversation

@shashankram

@shashankram shashankram commented Nov 5, 2026 •

Copy link
Copy Markdown

Backport #132 to release-1.0 so slow actor restores have more time before the router cancels them.

  • Default router parking and per-call API timeouts to 60s across all ingress routes.
  • Preserve the Helm timeout tests and the 5s parking E2E fixture.
  • Resolve documentation and test conflicts using the release branch's existing content.

Source: 4d8bf1ce7dadf584e9c9c0801ce8c18a76eb4308.

AI use: Codex applied the backport, resolved conflicts, reviewed the diff, and ran checks.

  • 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.
```

(cherry picked from commit 4d8bf1c)

Signed-off-by: Shashank Ram <shashank.ram@solo.io>
@shashankram
shashankram requested a review from EItanya November 5, 2026 17:09
@shashankram
shashankram merged commit d72e3a1 into release-1.0 Nov 5, 2026
6 checks passed
@shashankram
shashankram deleted the shashankram/backport-router-timeouts-1.0 branch November 5, 2026 17:19
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.

2 participants