Skip to content

Conversation

@TackAdam
Copy link
Collaborator

Description

[WIP] Jaeger support for traces

Issues Resolved

Screenshot

Testing the changes

Changelog

Check List

  • All tests pass
    • yarn test:jest
    • yarn test:jest_integration
  • New functionality includes testing.
  • New functionality has been documented.
  • Update CHANGELOG.md
  • Commits are signed per the DCO using --signoff

Signed-off-by: Adam Tackett <[email protected]>
@github-actions
Copy link
Contributor

ℹ️ Manual Changeset Creation Reminder

Please ensure manual commit for changeset file 10964.yml under folder changelogs/fragments to complete this PR.

If you want to use the available OpenSearch Changeset Bot App to avoid manual creation of changeset file you can install it in your forked repository following this link.

For more information about formatting of changeset files, please visit OpenSearch Auto Changeset and Release Notes Tool.

@codecov
Copy link

codecov bot commented Nov 22, 2025

Codecov Report

❌ Patch coverage is 50.80790% with 274 lines in your changes missing coverage. Please review.
✅ Project coverage is 60.74%. Comparing base (7038a73) to head (32bc144).

Files with missing lines Patch % Lines
...ces/trace_details/public/utils/span_data_utils.tsx 19.41% 116 Missing and 21 partials ⚠️
.../trace_details/public/traces/ppl_to_trace_hits.tsx 61.33% 33 Missing and 25 partials ⚠️
.../data_table/table_cell/trace_utils/trace_utils.tsx 30.55% 25 Missing ⚠️
...race_details/public/traces/ppl_resolve_helpers.tsx 78.26% 9 Missing and 11 partials ⚠️
...ins/explore/public/utils/trace_field_validation.ts 75.00% 6 Missing and 4 partials ⚠️
...ails/public/traces/span_tabs/span_overview_tab.tsx 47.05% 6 Missing and 3 partials ⚠️
...trace_details/public/utils/span_timerange_utils.ts 12.50% 6 Missing and 1 partial ⚠️
...ails/public/traces/span_tabs/span_metadata_tab.tsx 55.55% 4 Missing ⚠️
...races/trace_details/public/utils/grpc_constants.ts 66.66% 2 Missing ⚠️
...ic/components/data_table/table_cell/table_cell.tsx 0.00% 0 Missing and 1 partial ⚠️
... and 1 more
Additional details and impacted files
@@            Coverage Diff             @@
##             main   #10964      +/-   ##
==========================================
- Coverage   60.80%   60.74%   -0.06%     
==========================================
  Files        4532     4533       +1     
  Lines      122278   122789     +511     
  Branches    20502    20723     +221     
==========================================
+ Hits        74353    74592     +239     
- Misses      42686    42892     +206     
- Partials     5239     5305      +66     
Flag Coverage Δ
Linux_1 26.56% <ø> (ø)
Linux_2 38.92% <ø> (ø)
Linux_3 ?
Linux_4 33.82% <50.80%> (+0.07%) ⬆️
Windows_1 26.57% <ø> (ø)
Windows_2 38.90% <ø> (ø)
Windows_3 39.52% <ø> (+<0.01%) ⬆️
Windows_4 33.82% <50.80%> (+0.07%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

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

@coderabbitai
Copy link

coderabbitai bot commented Dec 1, 2025

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Comment @coderabbitai help to get the list of available commands and usage tips.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant