Skip to content

Fix: support Gateway API v1.3.0#7835

Closed
zhaohuabing wants to merge 3 commits intoenvoyproxy:mainfrom
zhaohuabing:fix-7709
Closed

Fix: support Gateway API v1.3.0#7835
zhaohuabing wants to merge 3 commits intoenvoyproxy:mainfrom
zhaohuabing:fix-7709

Conversation

@zhaohuabing
Copy link
Copy Markdown
Member

@zhaohuabing zhaohuabing commented Dec 29, 2025

This PR adds support for the Gateway API v1.3.0 experimental channel CRDs in the main branch.

The only CRD that requires backward compatibility handling is BackendTLSPolicy, which has graduated from v1alpha3 to the standard channel (v1).

It also adds e2e tests for Gateway API v1.3.0.

We do not add tests for v1.4.0, since there are no API changes between Gateway API v1.4.0 and v1.4.1—the latter only includes a bug fix related to the BackendTLSPolicy status.

Fixes: #7709

@zhaohuabing zhaohuabing requested a review from a team as a code owner December 29, 2025 02:23
@zhaohuabing zhaohuabing marked this pull request as draft December 29, 2025 02:23
@zhaohuabing zhaohuabing force-pushed the fix-7709 branch 4 times, most recently from c2cc53a to 8fb5f2b Compare December 29, 2025 02:35
@codecov
Copy link
Copy Markdown

codecov Bot commented Dec 29, 2025

Codecov Report

❌ Patch coverage is 24.59016% with 92 lines in your changes missing coverage. Please review.
✅ Project coverage is 72.66%. Comparing base (fe57c70) to head (3023af8).
⚠️ Report is 181 commits behind head on main.

Files with missing lines Patch % Lines
internal/provider/kubernetes/controller.go 36.11% 37 Missing and 9 partials ⚠️
internal/provider/kubernetes/indexers.go 2.38% 40 Missing and 1 partial ⚠️
internal/provider/kubernetes/predicates.go 37.50% 4 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #7835      +/-   ##
==========================================
- Coverage   72.77%   72.66%   -0.12%     
==========================================
  Files         235      235              
  Lines       35092    35181      +89     
==========================================
+ Hits        25538    25563      +25     
- Misses       7738     7802      +64     
  Partials     1816     1816              

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

@zhaohuabing zhaohuabing force-pushed the fix-7709 branch 7 times, most recently from 380e083 to a22307b Compare December 29, 2025 05:34
Signed-off-by: Huabing Zhao <zhaohuabing@gmail.com>
Signed-off-by: Huabing Zhao <zhaohuabing@gmail.com>
@zhaohuabing zhaohuabing force-pushed the fix-7709 branch 2 times, most recently from 59aa15d to 7db02f0 Compare December 29, 2025 06:45
Signed-off-by: Huabing Zhao <zhaohuabing@gmail.com>
@zhaohuabing
Copy link
Copy Markdown
Member Author

Close this in favor of #7523

@zhaohuabing zhaohuabing reopened this Dec 29, 2025
@zhaohuabing
Copy link
Copy Markdown
Member Author

zhaohuabing commented Dec 29, 2025

Reopen this as a draft, we might want to test Gateway v1.3.0 CRDs with EG v1.6 if we'd like to officially support multiple Gateway API versions.

@github-actions
Copy link
Copy Markdown
Contributor

This pull request has been automatically marked as stale because it has not had activity in the last 30 days. Please feel free to give a status update now, ping for review, when it's ready. Thank you for your contributions!

@github-actions github-actions Bot added stale and removed stale labels Jan 28, 2026
@zhaohuabing
Copy link
Copy Markdown
Member Author

Closing this as it can break the upgrade path from experimental to standard channel.

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.

Backend and BackendTLSPolicy stops working after v1.5.6

1 participant