Skip to content

[release/v1.7] Cherry pick for v1.7.1#8494

Merged
jukie merged 13 commits intoenvoyproxy:release/v1.7from
cnvergence:cherry-pick-v1.7.1
Mar 12, 2026
Merged

[release/v1.7] Cherry pick for v1.7.1#8494
jukie merged 13 commits intoenvoyproxy:release/v1.7from
cnvergence:cherry-pick-v1.7.1

Conversation

@cnvergence
Copy link
Copy Markdown
Member

@cnvergence cnvergence commented Mar 11, 2026

What type of PR is this?

What this PR does / why we need it:
Including fixes under https://github.com/envoyproxy/gateway/pulls?q=is%3Apr+label%3Acherrypick%2Frelease-v1.7.1+is%3Aclosed
Which issue(s) this PR fixes:

Fixes #8376

Release Notes: Yes/No

@netlify
Copy link
Copy Markdown

netlify bot commented Mar 11, 2026

Deploy Preview for cerulean-figolla-1f9435 ready!

Name Link
🔨 Latest commit 50c919f
🔍 Latest deploy log https://app.netlify.com/projects/cerulean-figolla-1f9435/deploys/69b1a65a37eb7b0008204935
😎 Deploy Preview https://deploy-preview-8494--cerulean-figolla-1f9435.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@cnvergence cnvergence changed the base branch from main to release/v1.7 March 11, 2026 17:29
* api: make ConnectionLimit.Value optional

Signed-off-by: Felipe Sabadini Facina <fsabadini@hotmail.com>

* release-notes: add entry for ConnectionLimit.Value optional

Signed-off-by: Felipe Sabadini Facina <fsabadini@hotmail.com>

* fix: add CEL rule to require value when closeDelay is set

Signed-off-by: Felipe Sabadini Facina <fsabadini@hotmail.com>

---------

Signed-off-by: Felipe Sabadini Facina <fsabadini@hotmail.com>
Signed-off-by: Karol Szwaj <karol.szwaj@gmail.com>
@cnvergence cnvergence force-pushed the cherry-pick-v1.7.1 branch 2 times, most recently from bc59118 to d25a677 Compare March 11, 2026 17:50
@cnvergence cnvergence marked this pull request as ready for review March 11, 2026 18:01
@cnvergence cnvergence requested a review from a team as a code owner March 11, 2026 18:01
cnvergence and others added 11 commits March 11, 2026 19:10
Signed-off-by: Karol Szwaj <karol.szwaj@gmail.com>
…ner (envoyproxy#8387)

* fix: aggregate xRoute/xPolicy statuses across GWCs in gateway-api runner

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

* polish

Signed-off-by: Huabing (Robin) Zhao <zhaohuabing@gmail.com>

* add e2e test

Signed-off-by: Huabing (Robin) Zhao <zhaohuabing@gmail.com>

* release note

Signed-off-by: Huabing (Robin) Zhao <zhaohuabing@gmail.com>

* truncate policy status & add tests

Signed-off-by: Huabing (Robin) Zhao <zhaohuabing@gmail.com>

* update

Signed-off-by: Huabing (Robin) Zhao <zhaohuabing@gmail.com>

* update

Signed-off-by: Huabing (Robin) Zhao <zhaohuabing@gmail.com>

---------

Signed-off-by: y-rabie <youssef.rabie@procore.com>
Signed-off-by: Huabing (Robin) Zhao <zhaohuabing@gmail.com>
Co-authored-by: y-rabie <youssef.rabie@procore.com>
Signed-off-by: Karol Szwaj <karol.szwaj@gmail.com>
revert unrelated changes

Signed-off-by: zirain <zirain2009@gmail.com>
Signed-off-by: Karol Szwaj <karol.szwaj@gmail.com>
…nvoyproxy#8321)

* add test for mixed managed and unmanaged Gateway parents

Signed-off-by: Huabing (Robin) Zhao <zhaohuabing@gmail.com>

* fix the policy status when the targeting routes have managed and unmanged Gateway parents

Signed-off-by: Huabing (Robin) Zhao <zhaohuabing@gmail.com>

* fix test

Signed-off-by: Huabing (Robin) Zhao <zhaohuabing@gmail.com>

---------

Signed-off-by: Huabing (Robin) Zhao <zhaohuabing@gmail.com>
Signed-off-by: Karol Szwaj <karol.szwaj@gmail.com>
)

Signed-off-by: Tejasriram Parvathaneni <tejaparvathaneni90@gmail.com>
Co-authored-by: Karol Szwaj <karol.szwaj@gmail.com>
Signed-off-by: Karol Szwaj <karol.szwaj@gmail.com>
…nvoyproxy#8186)

* fix: computeHosts doesn't work when listener and route both wildcard

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

* remove skipped tests

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

* Update internal/gatewayapi/helpers.go

Co-authored-by: Huabing (Robin) Zhao <zhaohuabing@gmail.com>
Signed-off-by: zirain <zirain2009@gmail.com>

---------

Signed-off-by: zirain <zirain2009@gmail.com>
Co-authored-by: Huabing (Robin) Zhao <zhaohuabing@gmail.com>
Signed-off-by: Karol Szwaj <karol.szwaj@gmail.com>
…ckendTrafficPolicies (envoyproxy#8210)

Signed-off-by: Rudrakh Panigrahi <rudrakh97@gmail.com>
Signed-off-by: Karol Szwaj <karol.szwaj@gmail.com>
Previously, using allowedRoutes/Same for an XListenerSet with an xRoute
in the same namespace would return an error. Now it properly allows
xRoutes from the same namespace.

Signed-off-by: Kris Hicks <khicks@nvidia.com>
Signed-off-by: Karol Szwaj <karol.szwaj@gmail.com>
* add test for multiple keys

Signed-off-by: Huabing (Robin) Zhao <zhaohuabing@gmail.com>

* revert secret transform

Signed-off-by: Huabing (Robin) Zhao <zhaohuabing@gmail.com>

---------

Signed-off-by: Huabing (Robin) Zhao <zhaohuabing@gmail.com>
Signed-off-by: Karol Szwaj <karol.szwaj@gmail.com>
Signed-off-by: Karol Szwaj <karol.szwaj@gmail.com>
Signed-off-by: Karol Szwaj <karol.szwaj@gmail.com>
@codecov
Copy link
Copy Markdown

codecov bot commented Mar 11, 2026

Codecov Report

❌ Patch coverage is 44.48529% with 151 lines in your changes missing coverage. Please review.
✅ Project coverage is 73.72%. Comparing base (19fa251) to head (b9cb9bc).
⚠️ Report is 1 commits behind head on release/v1.7.

Files with missing lines Patch % Lines
internal/gatewayapi/runner/runner.go 11.92% 133 Missing ⚠️
internal/gatewayapi/backendtrafficpolicy.go 67.56% 6 Missing and 6 partials ⚠️
internal/gatewayapi/extensionserverpolicy.go 86.66% 1 Missing and 1 partial ⚠️
internal/xds/translator/listener.go 71.42% 1 Missing and 1 partial ⚠️
internal/gatewayapi/filters.go 0.00% 0 Missing and 1 partial ⚠️
internal/gatewayapi/helpers.go 94.44% 1 Missing ⚠️

❌ Your patch status has failed because the patch coverage (44.48%) is below the target coverage (60.00%). You can increase the patch coverage or adjust the target coverage.

Additional details and impacted files
@@               Coverage Diff                @@
##           release/v1.7    #8494      +/-   ##
================================================
- Coverage         73.78%   73.72%   -0.07%     
================================================
  Files               241      241              
  Lines             36579    36718     +139     
================================================
+ Hits              26990    27069      +79     
- Misses             7681     7739      +58     
- Partials           1908     1910       +2     

☔ 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.
Signed-off-by: Karol Szwaj <karol.szwaj@gmail.com>
@jukie
Copy link
Copy Markdown
Contributor

jukie commented Mar 11, 2026

@cnvergence you'll want to add #8180 to fix the data race failure

@jukie
Copy link
Copy Markdown
Contributor

jukie commented Mar 11, 2026

/retest

@jukie jukie merged commit 4d13174 into envoyproxy:release/v1.7 Mar 12, 2026
32 of 39 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

8 participants