Skip to content

[release/v1.7] cherry-pick for v1.7.4#9120

Merged
zirain merged 8 commits into
envoyproxy:release/v1.7from
cnvergence:rel-1.7.4-cherrypick
Jun 4, 2026
Merged

[release/v1.7] cherry-pick for v1.7.4#9120
zirain merged 8 commits into
envoyproxy:release/v1.7from
cnvergence:rel-1.7.4-cherrypick

Conversation

@cnvergence

Copy link
Copy Markdown
Member

What type of PR is this?

What this PR does / why we need it:
https://github.com/envoyproxy/gateway/issues?q=state%3Aopen%20label%3Acherrypick%2Frelease-v1.7.4
Which issue(s) this PR fixes:

Fixes #

Release Notes: Yes/No

@cnvergence cnvergence requested a review from a team as a code owner May 29, 2026 16:04

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 22889011e3

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

Comment thread internal/gatewayapi/validate.go Outdated
@cnvergence cnvergence force-pushed the rel-1.7.4-cherrypick branch from 2288901 to f6f27db Compare May 29, 2026 16:21
@codecov

codecov Bot commented May 29, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 79.05983% with 49 lines in your changes missing coverage. Please review.
✅ Project coverage is 74.01%. Comparing base (523b97f) to head (41e7833).

Files with missing lines Patch % Lines
internal/gatewayapi/validate.go 74.03% 42 Missing and 5 partials ⚠️
internal/gatewayapi/listener.go 96.15% 2 Missing ⚠️
Additional details and impacted files
@@               Coverage Diff                @@
##           release/v1.7    #9120      +/-   ##
================================================
+ Coverage         74.00%   74.01%   +0.01%     
================================================
  Files               241      241              
  Lines             37026    37163     +137     
================================================
+ Hits              27400    27507     +107     
- Misses             7721     7750      +29     
- Partials           1905     1906       +1     

☔ 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.

@jukie

jukie commented May 30, 2026

Copy link
Copy Markdown
Contributor

/retest

cnvergence and others added 7 commits June 4, 2026 06:15
* exclude non-ready https

Signed-off-by: Karol Szwaj <[email protected]>

* prevent BoringSSL errors

Signed-off-by: Karol Szwaj <[email protected]>

* Add trailing newline to testdata PEMs

Signed-off-by: Karol Szwaj <[email protected]>

* add release notes

Signed-off-by: Karol Szwaj <[email protected]>

* use shallow copy instead of DeepCopy

Signed-off-by: Karol Szwaj <[email protected]>

* use maps.Clone for the data and split the PRs

Signed-off-by: Karol Szwaj <[email protected]>

---------

Signed-off-by: Karol Szwaj <[email protected]>
…oxy#8959)

* fix(xds): hot-reload xDS server cert in GatewayNamespaceMode

Signed-off-by: nguyenptk <[email protected]>

* extract loadServerTLSConfig helper and add tests

Signed-off-by: nguyenptk <[email protected]>

---------

Signed-off-by: nguyenptk <[email protected]>
Signed-off-by: Nguyên (Harry) <[email protected]>
Signed-off-by: Karol Szwaj <[email protected]>
…oxy#8962)

* fix: restore last transition time in merge status conditions

Signed-off-by: Rudrakh Panigrahi <[email protected]>

* add release note

Signed-off-by: Rudrakh Panigrahi <[email protected]>

---------

Signed-off-by: Rudrakh Panigrahi <[email protected]>
Signed-off-by: Karol Szwaj <[email protected]>
…same target (envoyproxy#9068)

* add passing testcase showing wildcard overriding sectioned backendTLSPolicy

Signed-off-by: AlecDiraimondo <[email protected]>

* fix(gatewayapi): prefer sectioned BackendTLSPolicy over wildcard for same target

When two BackendTLSPolicies target the same backend — one with sectionName
set, one without — the wildcard wins whenever it sorts earlier, since
policies are iterated in creationTimestamp order with no specificity
preference. The sectioned policy is silently dropped: never selected by
the translator, no status written.

Make getBackendTLSPolicy prefer policies with an explicit sectionName
match before falling back to wildcard matches. Matches the specificity
ordering SecurityPolicy already implements.

Refs envoyproxy#9035

Signed-off-by: AlecDiraimondo <[email protected]>

* update release notes

Signed-off-by: AlecDiraimondo <[email protected]>

* fixes from review: dedup helper, return early if backendTarget has no section specified

Signed-off-by: AlecDiraimondo <[email protected]>
Co-authored-by: Rudrakh Panigrahi <[email protected]>

---------

Signed-off-by: AlecDiraimondo <[email protected]>
Signed-off-by: Alec Diraimondo <[email protected]>
Co-authored-by: Rudrakh Panigrahi <[email protected]>
Signed-off-by: Karol Szwaj <[email protected]>
* skip invalid listener

Signed-off-by: zirain <[email protected]>

* fix specValid

Signed-off-by: zirain <[email protected]>

* nit

Signed-off-by: zirain <[email protected]>

* fix

Signed-off-by: zirain <[email protected]>

* MUST NOT pick one conflicting Listener as the winner

Signed-off-by: zirain <[email protected]>

* update

Signed-off-by: zirain <[email protected]>

---------

Signed-off-by: zirain <[email protected]>
Co-authored-by: Isaac Wilson <[email protected]>
Signed-off-by: Karol Szwaj <[email protected]>
Signed-off-by: Karol Szwaj <[email protected]>
Signed-off-by: Karol Szwaj <[email protected]>
@zirain zirain force-pushed the rel-1.7.4-cherrypick branch from 97e8831 to 2fad0b4 Compare June 3, 2026 22:18
Signed-off-by: Karol Szwaj <[email protected]>
Signed-off-by: zirain <[email protected]>
@zirain zirain force-pushed the rel-1.7.4-cherrypick branch from 2fad0b4 to 41e7833 Compare June 3, 2026 22:19
@zirain

zirain commented Jun 3, 2026

Copy link
Copy Markdown
Member

#8577 missed in #9152

@zirain zirain merged commit d318158 into envoyproxy:release/v1.7 Jun 4, 2026
63 of 72 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants