Skip to content

Remove deprecated jaeger-grpc monitor#6601

Merged
dmitryax merged 2 commits intomainfrom
remove-smartagent-jaeger
Aug 15, 2025
Merged

Remove deprecated jaeger-grpc monitor#6601
dmitryax merged 2 commits intomainfrom
remove-smartagent-jaeger

Conversation

@dmitryax
Copy link
Contributor

OTel Jaeger receiver should be used instead. To use it with the same port used by default by the jaeger-grpc monitor, apply the following config:

receivers:
  jaeger:
    protocols:
      grpc:
        endpoint: localhost:14250

@dmitryax dmitryax requested review from a team as code owners August 15, 2025 17:52
@dmitryax dmitryax force-pushed the remove-smartagent-jaeger branch from 1ca0cc3 to 965c17f Compare August 15, 2025 17:59
@codecov
Copy link

codecov bot commented Aug 15, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 34.21%. Comparing base (810d29d) to head (5475aa5).

❌ Your project check has failed because the head coverage (34.21%) is below the target coverage (40.00%). You can increase the head coverage or adjust the target coverage.

❗ There is a different number of reports uploaded between BASE (810d29d) and HEAD (5475aa5). Click for more details.

HEAD has 33 uploads less than BASE
Flag BASE (810d29d) HEAD (5475aa5)
36 3
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #6601      +/-   ##
==========================================
- Coverage   43.98%   34.21%   -9.77%     
==========================================
  Files         376      341      -35     
  Lines       26520    24589    -1931     
==========================================
- Hits        11665     8414    -3251     
- Misses      14027    15595    +1568     
+ Partials      828      580     -248     

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

Co-authored-by: Curtis Robert <crobert@splunk.com>
@dmitryax dmitryax merged commit 72c2bd2 into main Aug 15, 2025
233 of 234 checks passed
@dmitryax dmitryax deleted the remove-smartagent-jaeger branch August 15, 2025 19:44
@github-actions github-actions bot locked and limited conversation to collaborators Aug 15, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants