Skip to content

Commit 61ed2d1

Browse files
authored
Update SDK version (#7223)
Resolves #7221 Resolves #7222
1 parent 7ccf0bd commit 61ed2d1

File tree

12 files changed

+52
-57
lines changed

12 files changed

+52
-57
lines changed

bom-alpha/build.gradle.kts

+2-4
Original file line numberDiff line numberDiff line change
@@ -10,11 +10,9 @@ javaPlatform {
1010
allowDependencies()
1111
}
1212

13-
val otelVersion: String by project
14-
1513
dependencies {
16-
api(platform("io.opentelemetry:opentelemetry-bom:${otelVersion}"))
17-
api(platform("io.opentelemetry:opentelemetry-bom-alpha:${otelVersion}-alpha"))
14+
api(platform("io.opentelemetry:opentelemetry-bom"))
15+
api(platform("io.opentelemetry:opentelemetry-bom-alpha"))
1816
api(platform(project(":bom")))
1917
}
2018

bom/build.gradle.kts

+1-3
Original file line numberDiff line numberDiff line change
@@ -10,10 +10,8 @@ javaPlatform {
1010
allowDependencies()
1111
}
1212

13-
val otelVersion: String by project
14-
1513
dependencies {
16-
api(platform("io.opentelemetry:opentelemetry-bom:${otelVersion}"))
14+
api(platform("io.opentelemetry:opentelemetry-bom"))
1715
}
1816

1917
otelBom.projectFilter.set { it.findProperty("otel.stable") == "true" }

dependencyManagement/build.gradle.kts

+2-7
Original file line numberDiff line numberDiff line change
@@ -11,11 +11,6 @@ data class DependencySet(val group: String, val version: String, val modules: Li
1111
val dependencyVersions = hashMapOf<String, String>()
1212
rootProject.extra["versions"] = dependencyVersions
1313

14-
// this line is managed by .github/scripts/update-sdk-version.sh
15-
val otelVersion = "1.19.0"
16-
17-
rootProject.extra["otelVersion"] = otelVersion
18-
1914
// Need both BOM and groovy jars
2015
val groovyVersion = "4.0.6"
2116

@@ -34,8 +29,8 @@ val DEPENDENCY_BOMS = listOf(
3429
"com.fasterxml.jackson:jackson-bom:2.14.0",
3530
"com.google.guava:guava-bom:31.1-jre",
3631
"org.apache.groovy:groovy-bom:${groovyVersion}",
37-
"io.opentelemetry:opentelemetry-bom:${otelVersion}",
38-
"io.opentelemetry:opentelemetry-bom-alpha:${otelVersion}-alpha",
32+
"io.opentelemetry:opentelemetry-bom:1.20.1",
33+
"io.opentelemetry:opentelemetry-bom-alpha:1.20.1-alpha",
3934
"org.junit:junit-bom:5.9.1",
4035
"org.testcontainers:testcontainers-bom:1.17.6",
4136
"org.spockframework:spock-bom:2.3-groovy-4.0"

instrumentation/apache-camel-2.20/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/apachecamel/CamelPropagationUtil.java

+2-2
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,6 @@
99
import io.opentelemetry.context.Context;
1010
import io.opentelemetry.context.propagation.TextMapGetter;
1111
import io.opentelemetry.context.propagation.TextMapSetter;
12-
import io.opentelemetry.extension.aws.AwsXrayPropagator;
1312
import java.util.Collections;
1413
import java.util.Map;
1514
import org.apache.camel.Endpoint;
@@ -28,8 +27,9 @@ private static boolean isAwsPropagated(Endpoint endpoint) {
2827
return endpoint.getClass().getName().endsWith("SqsEndpoint");
2928
}
3029

30+
@SuppressWarnings("deprecation") // deprecated class to be updated once published in new location
3131
private static Context extractAwsPropagationParent(Map<String, Object> exchangeHeaders) {
32-
return AwsXrayPropagator.getInstance()
32+
return io.opentelemetry.extension.aws.AwsXrayPropagator.getInstance()
3333
.extract(
3434
Context.current(),
3535
Collections.singletonMap("X-Amzn-Trace-Id", exchangeHeaders.get("AWSTraceHeader")),

instrumentation/aws-lambda/aws-lambda-core-1.0/library/src/main/java/io/opentelemetry/instrumentation/awslambdacore/v1_0/internal/ParentContextExtractor.java

+2-2
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,6 @@
1111
import io.opentelemetry.api.trace.SpanContext;
1212
import io.opentelemetry.context.Context;
1313
import io.opentelemetry.context.propagation.TextMapGetter;
14-
import io.opentelemetry.extension.aws.AwsXrayPropagator;
1514
import java.util.Collections;
1615
import java.util.Locale;
1716
import java.util.Map;
@@ -54,8 +53,9 @@ private static Context fromHttpHeaders(
5453
// lower-case map getter used for extraction
5554
static final String AWS_TRACE_HEADER_PROPAGATOR_KEY = "x-amzn-trace-id";
5655

56+
@SuppressWarnings("deprecation") // deprecated class to be updated once published in new location
5757
public static Context fromXrayHeader(String parentHeader) {
58-
return AwsXrayPropagator.getInstance()
58+
return io.opentelemetry.extension.aws.AwsXrayPropagator.getInstance()
5959
.extract(
6060
// see BaseTracer#extract() on why we're using root() here
6161
Context.root(),

instrumentation/aws-lambda/aws-lambda-core-1.0/library/src/test/java/io/opentelemetry/instrumentation/awslambdacore/v1_0/internal/ApiGatewayProxyRequestTest.java

+4-2
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,6 @@
1515
import io.opentelemetry.api.GlobalOpenTelemetry;
1616
import io.opentelemetry.api.OpenTelemetry;
1717
import io.opentelemetry.context.propagation.ContextPropagators;
18-
import io.opentelemetry.extension.aws.AwsXrayPropagator;
1918
import io.opentelemetry.extension.trace.propagation.B3Propagator;
2019
import java.io.IOException;
2120
import java.io.InputStream;
@@ -42,11 +41,14 @@ public void shouldCreateNoopRequestIfNoPropagatorsSet() throws IOException {
4241
}
4342

4443
@Test
44+
@SuppressWarnings("deprecation") // deprecated class to be updated once published in new location
4545
public void shouldCreateNoopRequestIfXrayPropagatorsSet() throws IOException {
4646
// given
4747
InputStream mock = mock(InputStream.class);
4848
GlobalOpenTelemetry.set(
49-
OpenTelemetry.propagating(ContextPropagators.create(AwsXrayPropagator.getInstance())));
49+
OpenTelemetry.propagating(
50+
ContextPropagators.create(
51+
io.opentelemetry.extension.aws.AwsXrayPropagator.getInstance())));
5052
// when
5153
ApiGatewayProxyRequest created = ApiGatewayProxyRequest.forStream(mock);
5254
// then

instrumentation/aws-sdk/aws-sdk-1.11/library/src/main/java/io/opentelemetry/instrumentation/awssdk/v1_11/SqsParentContext.java

+2-2
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,6 @@
77

88
import io.opentelemetry.context.Context;
99
import io.opentelemetry.context.propagation.TextMapGetter;
10-
import io.opentelemetry.extension.aws.AwsXrayPropagator;
1110
import java.util.Collections;
1211
import java.util.Map;
1312

@@ -29,9 +28,10 @@ public String get(Map<String, String> map, String s) {
2928

3029
static final String AWS_TRACE_SYSTEM_ATTRIBUTE = "AWSTraceHeader";
3130

31+
@SuppressWarnings("deprecation") // deprecated class to be updated once published in new location
3232
static Context ofSystemAttributes(Map<String, String> systemAttributes) {
3333
String traceHeader = systemAttributes.get(AWS_TRACE_SYSTEM_ATTRIBUTE);
34-
return AwsXrayPropagator.getInstance()
34+
return io.opentelemetry.extension.aws.AwsXrayPropagator.getInstance()
3535
.extract(
3636
Context.root(),
3737
Collections.singletonMap("X-Amzn-Trace-Id", traceHeader),

instrumentation/aws-sdk/aws-sdk-1.11/library/src/main/java/io/opentelemetry/instrumentation/awssdk/v1_11/TracingRequestHandler.java

+3-2
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,6 @@
1111
import com.amazonaws.handlers.HandlerContextKey;
1212
import com.amazonaws.handlers.RequestHandler2;
1313
import io.opentelemetry.context.Context;
14-
import io.opentelemetry.extension.aws.AwsXrayPropagator;
1514
import io.opentelemetry.instrumentation.api.instrumenter.Instrumenter;
1615
import java.util.List;
1716
import javax.annotation.Nullable;
@@ -33,14 +32,16 @@ final class TracingRequestHandler extends RequestHandler2 {
3332
}
3433

3534
@Override
35+
@SuppressWarnings("deprecation") // deprecated class to be updated once published in new location
3636
public void beforeRequest(Request<?> request) {
3737
Context parentContext = Context.current();
3838
if (!requestInstrumenter.shouldStart(parentContext, request)) {
3939
return;
4040
}
4141
Context context = requestInstrumenter.start(parentContext, request);
4242

43-
AwsXrayPropagator.getInstance().inject(context, request, HeaderSetter.INSTANCE);
43+
io.opentelemetry.extension.aws.AwsXrayPropagator.getInstance()
44+
.inject(context, request, HeaderSetter.INSTANCE);
4445

4546
request.addHandlerContext(CONTEXT, context);
4647
}

instrumentation/aws-sdk/aws-sdk-2.2/library/src/main/java/io/opentelemetry/instrumentation/awssdk/v2_2/SqsParentContext.java

+2-2
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,6 @@
77

88
import io.opentelemetry.context.Context;
99
import io.opentelemetry.context.propagation.TextMapGetter;
10-
import io.opentelemetry.extension.aws.AwsXrayPropagator;
1110
import java.util.Collections;
1211
import java.util.Map;
1312

@@ -29,9 +28,10 @@ public String get(Map<String, String> map, String s) {
2928

3029
static final String AWS_TRACE_SYSTEM_ATTRIBUTE = "AWSTraceHeader";
3130

31+
@SuppressWarnings("deprecation") // deprecated class to be updated once published in new location
3232
static Context ofSystemAttributes(Map<String, String> systemAttributes) {
3333
String traceHeader = systemAttributes.get(AWS_TRACE_SYSTEM_ATTRIBUTE);
34-
return AwsXrayPropagator.getInstance()
34+
return io.opentelemetry.extension.aws.AwsXrayPropagator.getInstance()
3535
.extract(
3636
Context.root(),
3737
Collections.singletonMap("X-Amzn-Trace-Id", traceHeader),

instrumentation/aws-sdk/aws-sdk-2.2/library/src/main/java/io/opentelemetry/instrumentation/awssdk/v2_2/TracingExecutionInterceptor.java

+3-2
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,6 @@
99

1010
import io.opentelemetry.api.trace.Span;
1111
import io.opentelemetry.context.Scope;
12-
import io.opentelemetry.extension.aws.AwsXrayPropagator;
1312
import io.opentelemetry.instrumentation.api.instrumenter.Instrumenter;
1413
import io.opentelemetry.semconv.trace.attributes.SemanticAttributes;
1514
import java.util.ArrayList;
@@ -124,6 +123,7 @@ private static List<String> getAttributeNames(SdkRequest request) {
124123
}
125124

126125
@Override
126+
@SuppressWarnings("deprecation") // deprecated class to be updated once published in new location
127127
public SdkHttpRequest modifyHttpRequest(
128128
Context.ModifyHttpRequest context, ExecutionAttributes executionAttributes) {
129129
SdkHttpRequest httpRequest = context.httpRequest();
@@ -134,7 +134,8 @@ public SdkHttpRequest modifyHttpRequest(
134134
}
135135

136136
SdkHttpRequest.Builder builder = httpRequest.toBuilder();
137-
AwsXrayPropagator.getInstance().inject(otelContext, builder, RequestHeaderSetter.INSTANCE);
137+
io.opentelemetry.extension.aws.AwsXrayPropagator.getInstance()
138+
.inject(otelContext, builder, RequestHeaderSetter.INSTANCE);
138139
return builder.build();
139140
}
140141

instrumentation/spring/spring-boot-autoconfigure/src/main/java/io/opentelemetry/instrumentation/spring/autoconfigure/propagators/CompositeTextMapPropagatorFactory.java

+3-3
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,6 @@
1010
import io.opentelemetry.api.baggage.propagation.W3CBaggagePropagator;
1111
import io.opentelemetry.api.trace.propagation.W3CTraceContextPropagator;
1212
import io.opentelemetry.context.propagation.TextMapPropagator;
13-
import io.opentelemetry.extension.aws.AwsXrayPropagator;
1413
import io.opentelemetry.extension.trace.propagation.B3Propagator;
1514
import io.opentelemetry.extension.trace.propagation.JaegerPropagator;
1615
import io.opentelemetry.extension.trace.propagation.OtTracePropagator;
@@ -27,6 +26,7 @@ public final class CompositeTextMapPropagatorFactory {
2726
private static final Logger logger =
2827
Logger.getLogger(CompositeTextMapPropagatorFactory.class.getName());
2928

29+
@SuppressWarnings("deprecation") // deprecated class to be updated once published in new location
3030
static TextMapPropagator getCompositeTextMapPropagator(
3131
BeanFactory beanFactory, List<String> types) {
3232

@@ -70,8 +70,8 @@ static TextMapPropagator getCompositeTextMapPropagator(
7070
if (isOnClasspath("io.opentelemetry.extension.aws.AwsXrayPropagator")) {
7171
propagators.add(
7272
beanFactory
73-
.getBeanProvider(AwsXrayPropagator.class)
74-
.getIfAvailable(AwsXrayPropagator::getInstance));
73+
.getBeanProvider(io.opentelemetry.extension.aws.AwsXrayPropagator.class)
74+
.getIfAvailable(io.opentelemetry.extension.aws.AwsXrayPropagator::getInstance));
7575
}
7676
break;
7777
case "tracecontext":

0 commit comments

Comments
 (0)