Skip to content

[RUM 9903] Updating opentelemetry version#2786

Merged
satween merged 1 commit into
tvaleev/feature/RUM-9902-removing-unused-codefrom
tvaleev/feature/RUM-9903
Jul 14, 2025
Merged

[RUM 9903] Updating opentelemetry version#2786
satween merged 1 commit into
tvaleev/feature/RUM-9902-removing-unused-codefrom
tvaleev/feature/RUM-9903

Conversation

@satween

@satween satween commented Jul 11, 2025

Copy link
Copy Markdown
Contributor

What does this PR do?

  • Updating opentelemetry version up to 1.40.0
  • Removing separate opentelemetry for benchmark module

Motivation

  1. We was pretty behind (40+ versions) from the latest opentelemetry version.
  2. On the other hand SDK v3 gonna bring many changes and upgrading to 1.40.0 feels bit safer than to the most recent opentelemetry version as we already tested it in benchmark modules.
  3. And last point - we want to reduce the amount of dependencies used in project making benchmarking closer to what we ship to production, so from this moment there will be only one opentelemetry verison for whole project

Additional Notes

This branch targets to opentracing-opentelemetry migration branch to be sure that all test passed for the the most recent changes

Review checklist (to be filled by reviewers)

  • Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
  • Make sure you discussed the feature or bugfix with the maintaining team in an Issue
  • Make sure each commit and the PR mention the Issue number (cf the CONTRIBUTING doc)

@satween
satween requested review from a team as code owners July 11, 2025 11:22
@codecov-commenter

codecov-commenter commented Jul 11, 2025

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 70.41%. Comparing base (5a0abc0) to head (9169ab2).

Additional details and impacted files
@@                                Coverage Diff                                @@
##           tvaleev/feature/RUM-9902-removing-unused-code    #2786      +/-   ##
=================================================================================
+ Coverage                                          70.32%   70.41%   +0.10%     
=================================================================================
  Files                                                786      786              
  Lines                                              28715    28715              
  Branches                                            4823     4823              
=================================================================================
+ Hits                                               20191    20219      +28     
+ Misses                                              7169     7157      -12     
+ Partials                                            1355     1339      -16     

see 34 files with indirect coverage changes

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

@satween
satween force-pushed the tvaleev/feature/RUM-9902-removing-unused-code branch 2 times, most recently from e16333a to bee3881 Compare July 11, 2025 14:13

@jonathanmos jonathanmos left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM - I left some minor comments

@satween
satween force-pushed the tvaleev/feature/RUM-9903 branch from e9f9aaa to 9169ab2 Compare July 14, 2025 15:21
@satween
satween merged commit 0861b23 into tvaleev/feature/RUM-9902-removing-unused-code Jul 14, 2025
24 checks passed
@satween
satween deleted the tvaleev/feature/RUM-9903 branch July 14, 2025 16:47
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.

4 participants