[release/v1.5] cherry pick for v1.5.1 by zirain · Pull Request #6975 · envoyproxy/gateway

zirain · 2025-09-16T03:01:53Z

PR_NUMBERS:

cluster stat name: lowercase Kind Signed-off-by: Guy Daich <guy.daich@sap.com> Signed-off-by: zirain <zirain2009@gmail.com>

* fix!: fix envoy service cluster name for zone-aware routing Signed-off-by: y-rabie <youssef.rabie@procore.com> * extend e2e tests for zone aware routing Signed-off-by: y-rabie <youssef.rabie@procore.com> * extend unit tests for zone aware routing Signed-off-by: y-rabie <youssef.rabie@procore.com> --------- Signed-off-by: y-rabie <youssef.rabie@procore.com> Signed-off-by: zirain <zirain2009@gmail.com>

* conformance: update experimental test report Signed-off-by: zirain <zirain2009@gmail.com> * fix version Signed-off-by: zirain <zirain2009@gmail.com>

…oxy#6819) fix: match repository in image with port Signed-off-by: Windfarer <windfarer@gmail.com> Signed-off-by: zirain <zirain2009@gmail.com>

) * Actually update xdsIR with maxAcceptPerSocketEvent Signed-off-by: jukie <10012479+Jukie@users.noreply.github.com> * release note Signed-off-by: jukie <10012479+Jukie@users.noreply.github.com> * newline lint Signed-off-by: jukie <10012479+Jukie@users.noreply.github.com> --------- Signed-off-by: jukie <10012479+Jukie@users.noreply.github.com> Signed-off-by: zirain <zirain2009@gmail.com>

…t defined (envoyproxy#6847) * bugfix: fix the topologyInjectorDisabled and the local cluster was not defined. Signed-off-by: qicz <qiczzhu@gmail.com> * fix ut Signed-off-by: qicz <qiczzhu@gmail.com> * add topology-injector-enabled ut Signed-off-by: qicz <qiczzhu@gmail.com> * add release note Signed-off-by: qi <qiczzhu@gmail.com> --------- Signed-off-by: qicz <qiczzhu@gmail.com> Signed-off-by: qi <qiczzhu@gmail.com> Signed-off-by: zirain <zirain2009@gmail.com>

…ntime logger (envoyproxy#6846) * Update filters.go Signed-off-by: TomerJLevy <TomerJlevy@gmail.com> * add release notes Signed-off-by: TomerJLevy <TomerJlevy@gmail.com> --------- Signed-off-by: TomerJLevy <TomerJlevy@gmail.com> Signed-off-by: zirain <zirain2009@gmail.com>

* handle context errors as transient errors Signed-off-by: TomerJLevy <TomerJlevy@gmail.com> * add test cases Signed-off-by: TomerJLevy <TomerJlevy@gmail.com> * no need the new line Signed-off-by: TomerJLevy <TomerJlevy@gmail.com> * add release notes Signed-off-by: TomerJLevy <TomerJlevy@gmail.com> * Return the error as is Signed-off-by: TomerJLevy <TomerJlevy@gmail.com> * revert redundant changes Signed-off-by: TomerJLevy <TomerJlevy@gmail.com> * revert unrelated changes Signed-off-by: TomerJLevy <TomerJlevy@gmail.com> * revert more changes... Signed-off-by: TomerJLevy <TomerJlevy@gmail.com> --------- Signed-off-by: TomerJLevy <TomerJlevy@gmail.com> Co-authored-by: zirain <zirain2009@gmail.com> Signed-off-by: zirain <zirain2009@gmail.com>

…ss only. (envoyproxy#6838) Signed-off-by: zirain <zirain2009@gmail.com>

Signed-off-by: Shahar Harari <shahar.harari@sap.com> Signed-off-by: zirain <zirain2009@gmail.com>

Signed-off-by: Rudrakh Panigrahi <rudrakh97@gmail.com> Signed-off-by: zirain <zirain2009@gmail.com>

* improve targetRef selection for targetSelectors * only select refs in the same namespace as the policy Signed-off-by: Arko Dasgupta <arko@tetrate.io> * fix lint Signed-off-by: Arko Dasgupta <arko@tetrate.io> --------- Signed-off-by: Arko Dasgupta <arko@tetrate.io> Signed-off-by: zirain <zirain2009@gmail.com>

Signed-off-by: Rudrakh Panigrahi <rudrakh97@gmail.com> Signed-off-by: zirain <zirain2009@gmail.com>

* fix: rm incorrectly set exclusiveMaximum field in CRD * Also fix maximum value to 599 which includes 599 as a valid num Fixes: envoyproxy#6925 Signed-off-by: Arko Dasgupta <arko@tetrate.io> Signed-off-by: zirain <zirain2009@gmail.com>

* by default it should be unset which implies `Lax` Relates to envoyproxy#6347 Signed-off-by: Arko Dasgupta <arko@tetrate.io> Signed-off-by: zirain <zirain2009@gmail.com>

* Optimize pod cache Signed-off-by: jukie <10012479+Jukie@users.noreply.github.com> * release note Signed-off-by: jukie <10012479+Jukie@users.noreply.github.com> * Remove retry Signed-off-by: jukie <10012479+Jukie@users.noreply.github.com> * cleanup Signed-off-by: jukie <10012479+Jukie@users.noreply.github.com> --------- Signed-off-by: jukie <10012479+Jukie@users.noreply.github.com> Signed-off-by: Isaac <10012479+jukie@users.noreply.github.com> Co-authored-by: zirain <zirain2009@gmail.com> Signed-off-by: zirain <zirain2009@gmail.com>

* reduce deep copy in gateway-api layer * also fixed the DeepCopy implementation for ControllerResources which was performing a Shallow Copy resulting it lack of isolation b/w provider and gateway-api layer Relates to envoyproxy#6919 Signed-off-by: Arko Dasgupta <arko@tetrate.io> * fix lint Signed-off-by: Arko Dasgupta <arko@tetrate.io> --------- Signed-off-by: Arko Dasgupta <arko@tetrate.io> Signed-off-by: zirain <zirain2009@gmail.com>

…y#6949) Signed-off-by: Rudrakh Panigrahi <rudrakh97@gmail.com> Signed-off-by: zirain <zirain2009@gmail.com>

Signed-off-by: y-rabie <youssef.rabie@procore.com> Signed-off-by: zirain <zirain2009@gmail.com>

…erge (envoyproxy#6951) * fix: merge compression annotation Signed-off-by: sudipto baral <sudiptobaral.me@gmail.com> * test: add more compression merge test cases Signed-off-by: sudipto baral <sudiptobaral.me@gmail.com> --------- Signed-off-by: sudipto baral <sudiptobaral.me@gmail.com> Signed-off-by: zirain <zirain2009@gmail.com>

Signed-off-by: Shahar Harari <shahar.harari@sap.com> Signed-off-by: zirain <zirain2009@gmail.com>

chore: bump golang Signed-off-by: zirain <zirain2009@gmail.com>

…voyproxy#6972) Fixes: envoyproxy#6873 Signed-off-by: Arko Dasgupta <arko@tetrate.io> Co-authored-by: Jacob Neil Taylor <me@jacobtaylor.id.au> Signed-off-by: zirain <zirain2009@gmail.com>

Signed-off-by: zirain <zirain2009@gmail.com>

codecov · 2025-09-16T03:31:26Z

Codecov Report

❌ Patch coverage is 72.64957% with 96 lines in your changes missing coverage. Please review.
✅ Project coverage is 71.01%. Comparing base (c066ef5) to head (b05d90a).
⚠️ Report is 1 commits behind head on release/v1.5.

Files with missing lines	Patch %	Lines
internal/gatewayapi/contexts.go	64.97%	57 Missing and 5 partials ⚠️
internal/provider/kubernetes/controller.go	72.34%	10 Missing and 3 partials ⚠️
internal/provider/kubernetes/topology_injector.go	0.00%	7 Missing ⚠️
internal/provider/kubernetes/status.go	82.75%	5 Missing ⚠️
internal/provider/kubernetes/kubernetes.go	63.63%	4 Missing ⚠️
internal/provider/kubernetes/filters.go	0.00%	2 Missing ⚠️
internal/gatewayapi/backendtrafficpolicy.go	85.71%	1 Missing ⚠️
internal/gatewayapi/filters.go	0.00%	0 Missing and 1 partial ⚠️
internal/provider/kubernetes/indexers.go	0.00%	0 Missing and 1 partial ⚠️

Additional details and impacted files

@@               Coverage Diff                @@
##           release/v1.5    #6975      +/-   ##
================================================
- Coverage         71.10%   71.01%   -0.09%     
================================================
  Files               224      224              
  Lines             39600    39795     +195     
================================================
+ Hits              28156    28262     +106     
- Misses             9788     9865      +77     
- Partials           1656     1668      +12

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

zirain · 2025-09-16T04:26:46Z

unexcept error happen, something must be wrong.

…y#6820) * bench: adds APIToXDS bench & small opt Signed-off-by: Takeshi Yoneda <t.y.mathetake@gmail.com> * no refect goos: darwin goarch: arm64 pkg: github.com/envoyproxy/gateway/test/gobench cpu: Apple M1 Pro │ old.txt │ new.txt │ │ sec/op │ sec/op vs base │ GatewayAPItoXDS/small-10 881.2µ ± 7% 803.4µ ± 1% -8.82% (p=0.000 n=10) GatewayAPItoXDS/medium-10 4.130m ± 26% 2.959m ± 4% -28.36% (p=0.000 n=10) GatewayAPItoXDS/large-10 5.375 ± 2% 4.553 ± 1% -15.28% (p=0.000 n=10) geomean 26.94m 22.12m -17.90% │ old.txt │ new.txt │ │ B/op │ B/op vs base │ GatewayAPItoXDS/small-10 507.2Ki ± 0% 492.9Ki ± 0% -2.83% (p=0.000 n=10) GatewayAPItoXDS/medium-10 2.545Mi ± 7% 1.954Mi ± 2% -23.21% (p=0.000 n=10) GatewayAPItoXDS/large-10 2.832Gi ± 0% 2.831Gi ± 0% ~ (p=0.529 n=10) geomean 15.40Mi 13.97Mi -9.31% │ old.txt │ new.txt │ │ allocs/op │ allocs/op vs base │ GatewayAPItoXDS/small-10 8.328k ± 0% 8.017k ± 0% -3.73% (p=0.000 n=10) GatewayAPItoXDS/medium-10 39.45k ± 6% 29.74k ± 2% -24.60% (p=0.000 n=10) GatewayAPItoXDS/large-10 38.75M ± 0% 38.71M ± 0% -0.11% (p=0.000 n=10) geomean 233.5k 209.8k -10.16% Signed-off-by: Takeshi Yoneda <t.y.mathetake@gmail.com> * removes garbage Signed-off-by: Takeshi Yoneda <t.y.mathetake@gmail.com> * more tests Signed-off-by: Takeshi Yoneda <t.y.mathetake@gmail.com> * more tests Signed-off-by: Takeshi Yoneda <t.y.mathetake@gmail.com> --------- Signed-off-by: Takeshi Yoneda <t.y.mathetake@gmail.com>

zirain · 2025-09-16T04:46:34Z

unexcept error happen, something must be wrong.

missed to cherry pick #6820 .

Signed-off-by: zirain <zirain2009@gmail.com>

zirain · 2025-09-16T13:43:08Z

/retest

guydc and others added 24 commits September 16, 2025 10:59

fix: cluster stat name: lowercase Kind (envoyproxy#6780)

6e0fa64

cluster stat name: lowercase Kind Signed-off-by: Guy Daich <guy.daich@sap.com> Signed-off-by: zirain <zirain2009@gmail.com>

conformance: update experimental test report (envoyproxy#6782)

e596090

* conformance: update experimental test report Signed-off-by: zirain <zirain2009@gmail.com> * fix version Signed-off-by: zirain <zirain2009@gmail.com>

fix(api): image validation regex, support port in repository (envoypr…

7139088

…oxy#6819) fix: match repository in image with port Signed-off-by: Windfarer <windfarer@gmail.com> Signed-off-by: zirain <zirain2009@gmail.com>

bugfix: the controller cannot read the EnvoyProxy attached gatewaycla…

e40082e

…ss only. (envoyproxy#6838) Signed-off-by: zirain <zirain2009@gmail.com>

chore: fix CVE (envoyproxy#6903)

da4b9ff

Signed-off-by: Shahar Harari <shahar.harari@sap.com> Signed-off-by: zirain <zirain2009@gmail.com>

fix: nil pointer dereference in btp configmap indexer (envoyproxy#6921)

efbb804

Signed-off-by: Rudrakh Panigrahi <rudrakh97@gmail.com> Signed-off-by: zirain <zirain2009@gmail.com>

fix: suppress lua validation logs (envoyproxy#6929)

1a88162

Signed-off-by: Rudrakh Panigrahi <rudrakh97@gmail.com> Signed-off-by: zirain <zirain2009@gmail.com>

fix: rm Strict SameSite default (envoyproxy#6941)

a71027f

* by default it should be unset which implies `Lax` Relates to envoyproxy#6347 Signed-off-by: Arko Dasgupta <arko@tetrate.io> Signed-off-by: zirain <zirain2009@gmail.com>

fix: validation for grpc routes with extension ref filters (envoyprox…

b895143

…y#6949) Signed-off-by: Rudrakh Panigrahi <rudrakh97@gmail.com> Signed-off-by: zirain <zirain2009@gmail.com>

fix: cleanup dangling route status conditions (envoyproxy#6812)

8d8c7c6

Signed-off-by: y-rabie <youssef.rabie@procore.com> Signed-off-by: zirain <zirain2009@gmail.com>

fix: update distroless image to resolve glibc CVEs (envoyproxy#6953)

cacf995

Signed-off-by: Shahar Harari <shahar.harari@sap.com> Signed-off-by: zirain <zirain2009@gmail.com>

chore: bump golang to 1.24.7 (envoyproxy#6959)

3b3d11a

chore: bump golang Signed-off-by: zirain <zirain2009@gmail.com>

fix: Make sure proxy protocol filter is the first listener filter (en…

9581cff

…voyproxy#6972) Fixes: envoyproxy#6873 Signed-off-by: Arko Dasgupta <arko@tetrate.io> Co-authored-by: Jacob Neil Taylor <me@jacobtaylor.id.au> Signed-off-by: zirain <zirain2009@gmail.com>

release notes

8742fe0

Signed-off-by: zirain <zirain2009@gmail.com>

zirain requested a review from a team as a code owner September 16, 2025 03:01

zirain mentioned this pull request Sep 15, 2025

Release v1.5.1 #6957

Closed

11 tasks

zirain force-pushed the cp-1.5.1 branch from 02d4079 to 0549d4f Compare September 16, 2025 03:24

arkodg previously approved these changes Sep 16, 2025

View reviewed changes

zirain dismissed arkodg’s stale review via 8742fe0 September 16, 2025 04:16

zirain force-pushed the cp-1.5.1 branch from 0549d4f to 8742fe0 Compare September 16, 2025 04:16

zirain requested a review from arkodg September 16, 2025 04:46

arkodg previously approved these changes Sep 16, 2025

View reviewed changes

zirain enabled auto-merge (squash) September 16, 2025 05:08

zhaohuabing previously approved these changes Sep 16, 2025

View reviewed changes

[release/v1.5] 1.5.1 release-notes

8c133f0

Signed-off-by: zirain <zirain2009@gmail.com>

zirain dismissed stale reviews from zhaohuabing and arkodg via 8c133f0 September 16, 2025 06:22

fix benchmark job (envoyproxy#6979)

b05d90a

Signed-off-by: zirain <zirain2009@gmail.com>

arkodg approved these changes Sep 16, 2025

View reviewed changes

zirain requested a review from zhaohuabing September 16, 2025 13:59

zhaohuabing approved these changes Sep 16, 2025

View reviewed changes

zirain merged commit 3e5d1aa into envoyproxy:release/v1.5 Sep 16, 2025
43 of 45 checks passed

zirain deleted the cp-1.5.1 branch September 16, 2025 14:18

zirain mentioned this pull request Oct 1, 2025

Release 1.5.2 #7116

Closed

11 tasks

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[release/v1.5] cherry pick for v1.5.1#6975

[release/v1.5] cherry pick for v1.5.1#6975
zirain merged 27 commits intoenvoyproxy:release/v1.5from
zirain:cp-1.5.1

zirain commented Sep 16, 2025

Uh oh!

codecov bot commented Sep 16, 2025 •

edited

Loading

Uh oh!

zirain commented Sep 16, 2025

Uh oh!

zirain commented Sep 16, 2025

Uh oh!

zirain commented Sep 16, 2025

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

13 participants

Conversation

zirain commented Sep 16, 2025

Uh oh!

codecov bot commented Sep 16, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Codecov Report

Uh oh!

zirain commented Sep 16, 2025

Uh oh!

zirain commented Sep 16, 2025

Uh oh!

zirain commented Sep 16, 2025

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

13 participants

codecov bot commented Sep 16, 2025 •

edited

Loading