Skip to content

Comments

Add tracing to model provider and embedding provider#23

Merged
MohamedSabthar merged 10 commits intoballerina-platform:mainfrom
MohamedSabthar:tracing-support
Nov 4, 2025
Merged

Add tracing to model provider and embedding provider#23
MohamedSabthar merged 10 commits intoballerina-platform:mainfrom
MohamedSabthar:tracing-support

Conversation

@MohamedSabthar
Copy link
Member

@MohamedSabthar MohamedSabthar commented Oct 20, 2025

Purpose

Related to: ballerina-platform/ballerina-library#8341

Examples

Checklist

  • Linked to an issue
  • Test with Jaeger

@codecov
Copy link

codecov bot commented Nov 4, 2025

Codecov Report

❌ Patch coverage is 46.03175% with 68 lines in your changes missing coverage. Please review.
✅ Project coverage is 29.63%. Comparing base (32d6f50) to head (771ca57).
⚠️ Report is 11 commits behind head on main.

Files with missing lines Patch % Lines
ballerina/model-provider.bal 2.12% 46 Missing ⚠️
ballerina/embedding_provider.bal 54.54% 15 Missing ⚠️
ballerina/provider_utils.bal 84.78% 7 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #23      +/-   ##
==========================================
+ Coverage   28.22%   29.63%   +1.40%     
==========================================
  Files           5        5              
  Lines         751      847      +96     
  Branches      228      250      +22     
==========================================
+ Hits          212      251      +39     
- Misses        539      596      +57     

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

@MohamedSabthar MohamedSabthar merged commit d3f319e into ballerina-platform:main Nov 4, 2025
4 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