Skip to content

cleariy idle timeout docs#3701

Open
ramaraochavali wants to merge 3 commits intoistio:masterfrom
ramaraochavali:fix/idle_timeout_comments
Open

cleariy idle timeout docs#3701
ramaraochavali wants to merge 3 commits intoistio:masterfrom
ramaraochavali:fix/idle_timeout_comments

Conversation

@ramaraochavali
Copy link
Copy Markdown
Contributor

Signed-off-by: Rama Chavali <rama.rao@salesforce.com>
@ramaraochavali ramaraochavali requested a review from a team as a code owner May 3, 2026 09:20
@ramaraochavali ramaraochavali added the release-notes-none Indicates a PR that does not require release notes. label May 3, 2026
@istio-policy-bot
Copy link
Copy Markdown

🤔 🐛 You appear to be fixing a bug in Go code, yet your PR doesn't include updates to any test files. Did you forget to add a test?

Courtesy of your friendly test nag.

@istio-testing istio-testing added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label May 3, 2026
Signed-off-by: Rama Chavali <rama.rao@salesforce.com>
@istio-testing istio-testing added size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. and removed size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels May 4, 2026
// specified in a destination rule for the first weighted route is configured in the listener,
// which means also for all weighted routes.
// To set the idle timeout for downstream (inbound) connections, use the
// `idleTimeout` field in the proxy configuration (e.g., via the `proxy.istio.io/config`
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

is this correct? I thinbk it should be from ISTIO_META_IDLE_TIMEOUT

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

it actually has to be under metadata. I updated

Signed-off-by: Rama Chavali <rama.rao@salesforce.com>
@istio-testing istio-testing added size/S Denotes a PR that changes 10-29 lines, ignoring generated files. and removed size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels May 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release-notes-none Indicates a PR that does not require release notes. size/S Denotes a PR that changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

DestinationRule connectionPool.tcp.idleTimeout appears to be ignored in Istio 1.27.x (sidecar TCP)

4 participants