Skip to content

[v2.3.x] fix: include retry and timeouts in TrafficPolicy.Validate() (#14320)#14336

Merged
puertomontt merged 1 commit into
kgateway-dev:v2.3.xfrom
puertomontt:cherry-pick-e1a95bd-v2.3.x
Jun 30, 2026
Merged

[v2.3.x] fix: include retry and timeouts in TrafficPolicy.Validate() (#14320)#14336
puertomontt merged 1 commit into
kgateway-dev:v2.3.xfrom
puertomontt:cherry-pick-e1a95bd-v2.3.x

Conversation

@puertomontt

@puertomontt puertomontt commented Jun 30, 2026

Copy link
Copy Markdown
Contributor

Description

Backport of #14320 to the v2.3.x release branch.

TrafficPolicy.Validate() was not running the retry and timeouts sub-validators, so invalid retry/timeout configuration could slip through validation. This adds both validators to the aggregated list, matching the other policy sub-validators.

Cherry-picked from e1a95bd.

Change Type

/kind fix

Changelog

Fixed TrafficPolicy validation to include retry and timeout configuration.

Copilot AI review requested due to automatic review settings June 30, 2026 18:32
@gateway-bot gateway-bot added do-not-merge/description-invalid kind/fix Categorizes issue or PR as related to a bug. release-note labels Jun 30, 2026
@puertomontt
puertomontt enabled auto-merge June 30, 2026 18:38
@puertomontt
puertomontt added this pull request to the merge queue Jun 30, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

Merged via the queue into kgateway-dev:v2.3.x with commit b8f34ec Jun 30, 2026
32 of 37 checks passed
@puertomontt
puertomontt deleted the cherry-pick-e1a95bd-v2.3.x branch June 30, 2026 19:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kind/fix Categorizes issue or PR as related to a bug. release-note

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants