opentelemetry-collector icon indicating copy to clipboard operation
opentelemetry-collector copied to clipboard

Introduce MergeFrom for profiles

Open iblancasa opened this issue 1 month ago • 1 comments
trafficstars

Description

Link to tracking issue

This is part of https://github.com/open-telemetry/opentelemetry-collector/issues/13106 It introduces a new MergeFrom method, which allows merging to profiles.

Continuation fort #13588

iblancasa avatar Oct 06 '25 09:10 iblancasa

Codecov Report

:x: Patch coverage is 65.86667% with 128 lines in your changes missing coverage. Please review. :white_check_mark: Project coverage is 91.45%. Comparing base (5ec1466) to head (e2010f9). :warning: Report is 128 commits behind head on main.

Files with missing lines Patch % Lines
pdata/pprofile/merge.go 65.86% 87 Missing and 41 partials :warning:

:x: Your patch status has failed because the patch coverage (65.86%) is below the target coverage (95.00%). You can increase the patch coverage or adjust the target coverage.

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #13928      +/-   ##
==========================================
- Coverage   91.67%   91.45%   -0.22%     
==========================================
  Files         652      653       +1     
  Lines       42516    42891     +375     
==========================================
+ Hits        38977    39228     +251     
- Misses       2731     2815      +84     
- Partials      808      848      +40     

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

:rocket: New features to boost your workflow:
  • :snowflake: Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

codecov[bot] avatar Oct 06 '25 09:10 codecov[bot]

This PR was marked stale due to lack of activity. It will be closed in 14 days.

github-actions[bot] avatar Nov 07 '25 03:11 github-actions[bot]

We are working on this on other PRs.

iblancasa avatar Nov 07 '25 09:11 iblancasa