Skip to content

networking: add new budget_interval field to retry budget traffic policy#3716

Merged
istio-testing merged 2 commits into
istio:masterfrom
ericdbishop:retry-budget-interval-api
Jun 3, 2026
Merged

networking: add new budget_interval field to retry budget traffic policy#3716
istio-testing merged 2 commits into
istio:masterfrom
ericdbishop:retry-budget-interval-api

Conversation

@ericdbishop
Copy link
Copy Markdown
Member

Related: istio/istio#60389

Following the addition of the budget_interval parameter in the upstream Envoy retry budget configuration, exposing it in the retry budget traffic policy so that we can translate the BudgetDetails.Interval field of BackendTrafficPolicy into Istio CRD configuration.

Signed-off-by: Eric Bishop <ericbish.dev@gmail.com>
@ericdbishop ericdbishop requested a review from a team as a code owner May 29, 2026 14:48
@istio-testing istio-testing added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label May 29, 2026
@zirain zirain added the release-notes-none Indicates a PR that does not require release notes. label May 29, 2026
@zirain
Copy link
Copy Markdown
Member

zirain commented May 29, 2026

/test release-notes

Signed-off-by: Eric Bishop <ericbish.dev@gmail.com>
@keithmattix keithmattix added the do-not-merge/hold Block automatic merging of a PR. label Jun 1, 2026
Copy link
Copy Markdown
Contributor

@keithmattix keithmattix left a comment

Choose a reason for hiding this comment

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

Seems fairly straightforward to me. Approving with a hold for lazy consensus; remove if a few days pass or you get another approval from a TOC member

@ramaraochavali ramaraochavali removed the do-not-merge/hold Block automatic merging of a PR. label Jun 3, 2026
@istio-testing istio-testing merged commit 7eca3b2 into istio:master Jun 3, 2026
5 checks passed
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.

5 participants