Skip to content

Update the OpenTelemetry SDK version to 1.63.0#18911

Merged
laurit merged 15 commits into
mainfrom
otelbot/update-opentelemetry-sdk-to-1.63.0
Jun 10, 2026
Merged

Update the OpenTelemetry SDK version to 1.63.0#18911
laurit merged 15 commits into
mainfrom
otelbot/update-opentelemetry-sdk-to-1.63.0

Conversation

@otelbot

@otelbot otelbot Bot commented Jun 5, 2026

Copy link
Copy Markdown
Contributor

Update the OpenTelemetry SDK version to 1.63.0.

@otelbot
otelbot Bot requested a review from a team as a code owner June 5, 2026 15:49
Copilot AI review requested due to automatic review settings June 5, 2026 15:49

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

OpenTelemetry 1.63 moved declarative config model classes under sdk.autoconfigure.declarativeconfig.model and moved InstrumentationUtil under api.impl, while removing the extended log testing APIs used by compatibility helpers. Update those package references, keep the InstrumentationUtil javaagent bridge matching both old and new application class names, and convert incubating extended log attributes into the current Attributes/Value representation.

Validation: .\gradlew.bat :sdk-autoconfigure-support:compileJava :instrumentation-api:test --tests SpanSuppressionStrategyTest :javaagent-tooling:test --tests SpanLoggingCustomizerProviderTest --tests ResourceCustomizerProviderTest :instrumentation:spring:spring-boot-autoconfigure:test --tests '*SpanLoggingCustomizerProviderTest' :declarative-config-bridge:test :testing-common:compileJava :testing:agent-exporter:compileJava :instrumentation:opentelemetry-api:opentelemetry-api-1.50:javaagent:incubatorTest --tests '*LoggerTest'

Co-authored-by: Copilot <[email protected]>

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 27 out of 27 changed files in this pull request and generated 1 comment.

trask and others added 9 commits June 6, 2026 15:47
…telemetry-sdk-to-1.63.0

# Conflicts:
#	testing-common/src/main/java/io/opentelemetry/instrumentation/testing/internal/TelemetryConverter.java
Co-authored-by: Copilot Autofix powered by AI <[email protected]>
The 1.47 incubator logs bridge referenced an API shape that is only valid in later incubator versions, which caused muzzle failures across unrelated test shards and pulled in unavailable SDK testing internals. Remove the unsupported 1.47 logs bridge, keep the 1.47 module focused on trace/metrics, stop forcing the old SDK testing artifact, and apply the Spotless-required comment wrap.

Validation: .\gradlew.bat :instrumentation:opentelemetry-api:opentelemetry-api-1.27:javaagent:spotlessCheck :instrumentation:opentelemetry-api:opentelemetry-api-1.47:javaagent:spotlessCheck :instrumentation:opentelemetry-api:opentelemetry-api-1.50:javaagent:spotlessCheck :instrumentation:opentelemetry-api:opentelemetry-api-1.47:javaagent:test :instrumentation:opencensus-shim:testing:test

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 48 out of 48 changed files in this pull request and generated 2 comments.

Comment on lines +383 to +393
builder =
invoke(
builder, "setSeverityText", new Class<?>[] {String.class}, logRecord.getSeverityText());
builder =
invoke(builder, "setEventName", new Class<?>[] {String.class}, logRecord.getEventName());
builder =
invoke(
builder,
"setBodyValue",
new Class<?>[] {Value.class},
getBodyValue(logRecord.getBody()));
Comment thread settings.gradle.kts
include(":instrumentation:opentelemetry-api:opentelemetry-api-1.57:javaagent")
include(":instrumentation:opentelemetry-api:opentelemetry-api-1.59:javaagent")
include(":instrumentation:opentelemetry-api:opentelemetry-api-1.61:testing")
include(":instrumentation:opentelemetry-api:opentelemetry-api-1.63:javaagent")

@Override
@CanIgnoreReturnValue
public application.io.opentelemetry.api.logs.LogRecordBuilder setException(Throwable throwable) {

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

was really added in 1.60.0

import io.opentelemetry.api.logs.LogRecordBuilder;
import io.opentelemetry.javaagent.instrumentation.opentelemetryapi.v1_42.logs.ApplicationLogRecordBuilder142;

public class ApplicationLogRecordBuilder147Incubator extends ApplicationLogRecordBuilder142

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I guess this got removed because it was too annoying to test? I think we might need a better strategy. Maybe shading something in would help? Or perhaps allow tests to provide custom conversions for TelemetryConverter.

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.

@laurit
laurit merged commit 515d1d8 into main Jun 10, 2026
98 checks passed
@laurit
laurit deleted the otelbot/update-opentelemetry-sdk-to-1.63.0 branch June 10, 2026 07:42
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.

3 participants