Skip to content

Update otel version - #44715

Open
NipunaMadhushan wants to merge 2 commits into
ballerina-platform:masterfrom
NipunaMadhushan:update-otel-versions-master
Open

Update otel version#44715
NipunaMadhushan wants to merge 2 commits into
ballerina-platform:masterfrom
NipunaMadhushan:update-otel-versions-master

Conversation

@NipunaMadhushan

@NipunaMadhushan NipunaMadhushan commented Sep 9, 2026

Copy link
Copy Markdown
Contributor

Purpose

$subject

Eventhough, io.opentelemetry:opentelemetry-common is included inside io.opentelemetry:opentelemetry-context package in 1.32.0 version, it has become a separate package with 1.65.0 version.

Fixes #44716

Approach

Describe how you are implementing the solutions along with the design details.

Samples

Provide high-level details about the samples related to this feature.

Remarks

List any other known issues, related PRs, TODO items, or any other notes related to the PR.

Check List

  • Read the Contributing Guide
  • Updated Change Log
  • Checked Tooling Support (#)
  • Added necessary tests
    • Unit Tests
    • Spec Conformance Tests
    • Integration Tests
    • Ballerina By Example Tests
  • Increased Test Coverage
  • Added necessary documentation
    • API documentation
    • Module documentation in Module.md files
    • Ballerina By Examples

Summary

  • Updated OpenTelemetry dependencies to version 1.65.0.
  • Added the OpenTelemetry common library to version configuration and Ballerina runtime distribution dependencies.

@coderabbitai

coderabbitai Bot commented Sep 9, 2026

Copy link
Copy Markdown

Review Change StackReview Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Advanced

Run ID: 2fdac523-156d-4a9f-af1a-04bb04a2169f

📥 Commits

Reviewing files that changed from the base of the PR and between 80fce8c and 740e644.

📒 Files selected for processing (1)
  • gradle/libs.versions.toml

Included review availability: Your plan provides up to 8 included reviews per hour; 6 remain after this review.


📝 Walkthrough

Walkthrough

The build updates OpenTelemetry dependencies from 1.63.0 to 1.65.0 and adds the OpenTelemetry Common library to the Ballerina runtime distribution.

Changes

OpenTelemetry runtime distribution

Layer / File(s) Summary
Dependency catalog and runtime wiring
gradle/libs.versions.toml, bvm/ballerina-rt/build.gradle
OpenTelemetry API, context, common, SDK testing, and SDK trace dependencies are updated to 1.65.0. The common library is added to the runtime distribution.

Priority: ➖ Normal

Estimated code review effort: 1 (Trivial) | ~5 minutes

Severity of issue fixed: Medium

Merge Risk: ⚪ Minimal · up to 740e6

This updates OpenTelemetry dependencies and packages the common library with the runtime distribution. The dependency set is aligned, with no identified merge-blocking impact.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The description includes the linked issue, but the purpose contains the unresolved "$subject" placeholder. The Approach, Samples, and Remarks sections remain template text, and the checklist is not co… Replace "$subject" with a specific purpose. Complete the Approach, Samples, and Remarks sections, and update each checklist item to reflect the work performed.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly identifies the main change: updating the OpenTelemetry version.
Linked Issues check ✅ Passed The changes update the OpenTelemetry version entries to 1.65.0 and add the required OpenTelemetry common dependency. These changes satisfy issue #44716.
Out of Scope Changes check ✅ Passed The dependency declaration and version updates are directly related to the OpenTelemetry upgrade requested by issue #44716. No unrelated changes are identified.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Full details: Description check

Explanation

The description includes the linked issue, but the purpose contains the unresolved "$subject" placeholder. The Approach, Samples, and Remarks sections remain template text, and the checklist is not completed.

  • Fix all pre-merge checks with AI
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

A rabbit checks the version book
Common joins the runtime nook
The traces hop in tidy rows
At sixty-five, the package grows
Ears up for cleaner builds
Moonlight fills the dependency fields

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

@codecov

codecov Bot commented Sep 9, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 75.13%. Comparing base (3b82c03) to head (740e644).
⚠️ Report is 145 commits behind head on master.

Additional details and impacted files
@@             Coverage Diff              @@
##             master   #44715      +/-   ##
============================================
+ Coverage     75.03%   75.13%   +0.10%     
- Complexity    58724    59181     +457     
============================================
  Files          3601     3615      +14     
  Lines        227211   228909    +1698     
  Branches      29595    29809     +214     
============================================
+ Hits         170482   171987    +1505     
- Misses        47223    47276      +53     
- Partials       9506     9646     +140     

☔ View full report in Codecov by Harness.
📢 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.

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.

[Task]: Update opentelemetry versions to latest

2 participants