Skip to content

Removes reflection from RouteContext to reduce allocations#6820

Merged
arkodg merged 5 commits into
envoyproxy:mainfrom
mathetake:translate
Sep 9, 2025
Merged

Removes reflection from RouteContext to reduce allocations#6820
arkodg merged 5 commits into
envoyproxy:mainfrom
mathetake:translate

more tests

98248bd
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project succeeded Aug 20, 2025 in 1s

71.03% (target 60.00%)

View this Pull Request on Codecov

71.03% (target 60.00%)

Details

Codecov Report

❌ Patch coverage is 72.86432% with 54 lines in your changes missing coverage. Please review.
✅ Project coverage is 71.03%. Comparing base (c19352e) to head (98248bd).

Files with missing lines Patch % Lines
internal/gatewayapi/contexts.go 69.49% 49 Missing and 5 partials ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #6820      +/-   ##
==========================================
- Coverage   71.04%   71.03%   -0.01%     
==========================================
  Files         225      225              
  Lines       39810    39912     +102     
==========================================
+ Hits        28282    28352      +70     
- Misses       9859     9891      +32     
  Partials     1669     1669              

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