Skip to content

Conversation

@codefromthecrypt
Copy link
Contributor

Description

There is no issue, but we were missing tests for tracing and metrics authentication. This is needed for cloud environments like Elastic.

Related Issues/PRs (if applicable)

#1150

@codefromthecrypt codefromthecrypt requested a review from a team as a code owner September 23, 2025 08:31
@codecov-commenter
Copy link

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 79.64%. Comparing base (9e9fba9) to head (d435e5a).

❌ Your project status has failed because the head coverage (79.64%) is below the target coverage (86.00%). You can increase the head coverage or adjust the target coverage.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1225      +/-   ##
==========================================
- Coverage   79.68%   79.64%   -0.04%     
==========================================
  Files          90       90              
  Lines       10247    10247              
==========================================
- Hits         8165     8161       -4     
- Misses       1719     1724       +5     
+ Partials      363      362       -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.

@mathetake mathetake merged commit ff08d2e into envoyproxy:main Sep 23, 2025
28 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.

3 participants