Skip to content

revist TLSRoute conformance test#8345

Merged
zirain merged 1 commit into
envoyproxy:mainfrom
zirain:tlsroute-conformance-test
Feb 26, 2026
Merged

revist TLSRoute conformance test#8345
zirain merged 1 commit into
envoyproxy:mainfrom
zirain:tlsroute-conformance-test

Conversation

@zirain
Copy link
Copy Markdown
Member

@zirain zirain commented Feb 25, 2026

No description provided.

Signed-off-by: zirain <zirain2009@gmail.com>
@zirain zirain requested a review from a team as a code owner February 25, 2026 09:17
@netlify
Copy link
Copy Markdown

netlify Bot commented Feb 25, 2026

Deploy Preview for cerulean-figolla-1f9435 ready!

Name Link
🔨 Latest commit c793c58
🔍 Latest deploy log https://app.netlify.com/projects/cerulean-figolla-1f9435/deploys/699ebe290f3aed00084f1bd9
😎 Deploy Preview https://deploy-preview-8345--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.

Comment thread test/conformance/suite.go
@@ -22,10 +22,6 @@ func SkipTests(gatewayNamespaceMode bool) []suite.ConformanceTest {
tests.TLSRouteHostnameIntersection,
tests.TLSRouteInvalidNoMatchingListener,
tests.TLSRouteInvalidNoMatchingListenerHostname,
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

These are not included?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

@zirain zirain merged commit 2f13456 into envoyproxy:main Feb 26, 2026
75 of 80 checks passed
@zirain zirain deleted the tlsroute-conformance-test branch February 26, 2026 01:47
@codecov
Copy link
Copy Markdown

codecov Bot commented Feb 26, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 73.79%. Comparing base (26fa4e6) to head (c793c58).
⚠️ Report is 5 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #8345      +/-   ##
==========================================
+ Coverage   73.73%   73.79%   +0.06%     
==========================================
  Files         241      241              
  Lines       37077    37077              
==========================================
+ Hits        27337    27361      +24     
+ Misses       7796     7773      -23     
+ Partials     1944     1943       -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.

antonio-mazzini pushed a commit to antonio-mazzini/gateway that referenced this pull request Mar 5, 2026
Signed-off-by: zirain <zirain2009@gmail.com>
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.

2 participants