Skip to content

feat: use envoy matcher api for routes - #8372

Closed
rudrakhp wants to merge 1 commit into
envoyproxy:mainfrom
rudrakhp:sublinear_route_matcher
Closed

feat: use envoy matcher api for routes#8372
rudrakhp wants to merge 1 commit into
envoyproxy:mainfrom
rudrakhp:sublinear_route_matcher

Conversation

@rudrakhp

Copy link
Copy Markdown
Member

What type of PR is this?

What this PR does / why we need it:

Which issue(s) this PR fixes:

Fixes #

Release Notes: Yes/No

@netlify

netlify Bot commented Feb 28, 2026

Copy link
Copy Markdown

Deploy Preview for cerulean-figolla-1f9435 canceled.

Name Link
🔨 Latest commit b3854e4
🔍 Latest deploy log https://app.netlify.com/projects/cerulean-figolla-1f9435/deploys/69a345eb7818620008d6cdfe

@codecov

codecov Bot commented Feb 28, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 79.36508% with 39 lines in your changes missing coverage. Please review.
✅ Project coverage is 74.00%. Comparing base (bf0f05b) to head (b3854e4).
⚠️ Report is 170 commits behind head on main.

Files with missing lines Patch % Lines
internal/xds/translator/route_matcher.go 80.51% 21 Missing and 9 partials ⚠️
internal/xds/translator/translator.go 74.28% 6 Missing and 3 partials ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #8372      +/-   ##
==========================================
+ Coverage   73.85%   74.00%   +0.14%     
==========================================
  Files         242      243       +1     
  Lines       37061    37346     +285     
==========================================
+ Hits        27372    27637     +265     
- Misses       7761     7768       +7     
- Partials     1928     1941      +13     

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

@rudrakhp
rudrakhp force-pushed the sublinear_route_matcher branch 2 times, most recently from 95731b1 to 3d92c37 Compare February 28, 2026 14:59
Signed-off-by: Rudrakh Panigrahi <rudrakh97@gmail.com>
@rudrakhp
rudrakhp force-pushed the sublinear_route_matcher branch from 3d92c37 to b3854e4 Compare February 28, 2026 19:45
@zhaohuabing

zhaohuabing commented Mar 6, 2026

Copy link
Copy Markdown
Member

Hi @rudrakhp thanks for the work on improving routing performance! Have we discussed this in a community meeting? This could be a breaking change for EEP and the extension server.

@github-actions

github-actions Bot commented Apr 5, 2026

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 the stale label Apr 5, 2026
@github-actions github-actions Bot closed this Apr 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants