Skip to content

Update OpenTelemetry to 1.67.0 with R8 resource fix - #936

Merged
LikeTheSalad merged 1 commit into
mainfrom
fix/retain-otel-resource-entities
Oct 14, 2026
Merged

LikeTheSalad merged 1 commit into
mainfrom
fix/retain-otel-resource-entities

Conversation

@LikeTheSalad

Copy link
Copy Markdown
Contributor

Supersedes #934

Summary

Update OpenTelemetry to 1.67.0 while preserving reflective resource access in minified apps.

Changes

  • Update OpenTelemetry core to 1.67.0 and the incubator API to 1.67.0-alpha.
  • Keep Resource.getEntities() available to OTLP exporters after R8 shrinking.

@github-actions

Copy link
Copy Markdown

🤖 GitHub comments

Just comment with:

  • run docs-build : Re-trigger the docs validation. (use unformatted text in the comment!)

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟢 Approval recommended

The dependency versions are aligned, and the targeted keep rule is exercised by the existing minified integration export path.

0 open findings

What changed in this PR

Updates OpenTelemetry to 1.67.0 while preserving OTLP resource export in R8-minified applications.

Changes:

  • Upgrades OpenTelemetry core and incubator dependencies.
  • Adds an R8 rule preserving reflective Resource.getEntities() access.
File Description
gradle/​libs.versions.toml Updates OpenTelemetry dependency versions.
shared-rules.pro Preserves the resource entity accessor during shrinking.

🧠 Review effort: Balanced


💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@LikeTheSalad
LikeTheSalad marked this pull request as ready for review October 14, 2026 09:50
@LikeTheSalad
LikeTheSalad requested a review from a team October 14, 2026 09:50
@LikeTheSalad
LikeTheSalad merged commit 9a0a0cb into main Oct 14, 2026
16 checks passed
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.

3 participants