Deprecate multi argument InstrumentationScopeInfo create method (open…

…-telemetry#4710) * Deprecate multi argument InstrumentationScopeInfo create method * PR feedback
qixiaogang · Aug 24, 2022 · 2bc88f4 · 2bc88f4
1 parent 8445642
commit 2bc88f4
Show file tree

Hide file tree

Showing 52 changed files with 262 additions and 166 deletions.
diff --git a/docs/apidiffs/current_vs_latest/opentelemetry-sdk-common.txt b/docs/apidiffs/current_vs_latest/opentelemetry-sdk-common.txt
@@ -2,6 +2,8 @@ Comparing source compatibility of  against
 **** MODIFIED CLASS: PUBLIC ABSTRACT io.opentelemetry.sdk.common.InstrumentationScopeInfo  (not serializable)
 	===  CLASS FILE FORMAT VERSION: 52.0 <- 52.0
 	+++  NEW METHOD: PUBLIC(+) STATIC(+) io.opentelemetry.sdk.common.InstrumentationScopeInfoBuilder builder(java.lang.String)
+	===  UNCHANGED METHOD: PUBLIC STATIC io.opentelemetry.sdk.common.InstrumentationScopeInfo create(java.lang.String, java.lang.String, java.lang.String)
+		+++  NEW ANNOTATION: java.lang.Deprecated
 	+++* NEW METHOD: PUBLIC(+) ABSTRACT(+) io.opentelemetry.api.common.Attributes getAttributes()
 +++  NEW CLASS: PUBLIC(+) FINAL(+) io.opentelemetry.sdk.common.InstrumentationScopeInfoBuilder  (not serializable)
 	+++  CLASS FILE FORMAT VERSION: 52.0 <- n.a.

diff --git a/...t/src/test/java/io/opentelemetry/exporter/jaeger/thrift/JaegerThriftSpanExporterTest.java b/...t/src/test/java/io/opentelemetry/exporter/jaeger/thrift/JaegerThriftSpanExporterTest.java
@@ -83,7 +83,9 @@ void testExport() throws SenderException, UnknownHostException {
             .setTotalRecordedLinks(0)
             .setTotalRecordedEvents(0)
             .setInstrumentationScopeInfo(
-                InstrumentationScopeInfo.create("io.opentelemetry.auto", "1.0.0", null))
+                InstrumentationScopeInfo.builder("io.opentelemetry.auto")
+                    .setVersion("1.0.0")
+                    .build())
             .setResource(
                 Resource.create(
                     Attributes.of(
@@ -158,7 +160,9 @@ void testExportMultipleResources() throws SenderException, UnknownHostException
             .setTotalRecordedLinks(0)
             .setTotalRecordedEvents(0)
             .setInstrumentationScopeInfo(
-                InstrumentationScopeInfo.create("io.opentelemetry.auto", "1.0.0", null))
+                InstrumentationScopeInfo.builder("io.opentelemetry.auto")
+                    .setVersion("1.0.0")
+                    .build())
             .setResource(
                 Resource.create(
                     Attributes.of(
@@ -181,7 +185,9 @@ void testExportMultipleResources() throws SenderException, UnknownHostException
             .setTotalRecordedLinks(0)
             .setTotalRecordedEvents(0)
             .setInstrumentationScopeInfo(
-                InstrumentationScopeInfo.create("io.opentelemetry.auto", "1.0.0", null))
+                InstrumentationScopeInfo.builder("io.opentelemetry.auto")
+                    .setVersion("1.0.0")
+                    .build())
             .setResource(
                 Resource.create(
                     Attributes.of(

diff --git a/...ers/jaeger/src/test/java/io/opentelemetry/exporter/jaeger/JaegerGrpcSpanExporterTest.java b/...ers/jaeger/src/test/java/io/opentelemetry/exporter/jaeger/JaegerGrpcSpanExporterTest.java
@@ -126,7 +126,9 @@ void testExport() throws Exception {
             .setTotalRecordedLinks(0)
             .setTotalRecordedEvents(0)
             .setInstrumentationScopeInfo(
-                InstrumentationScopeInfo.create("io.opentelemetry.auto", "1.0.0", null))
+                InstrumentationScopeInfo.builder("io.opentelemetry.auto")
+                    .setVersion("1.0.0")
+                    .build())
             .setResource(
                 Resource.create(
                     Attributes.of(
@@ -177,7 +179,9 @@ void testExportMultipleResources() throws Exception {
             .setTotalRecordedLinks(0)
             .setTotalRecordedEvents(0)
             .setInstrumentationScopeInfo(
-                InstrumentationScopeInfo.create("io.opentelemetry.auto", "1.0.0", null))
+                InstrumentationScopeInfo.builder("io.opentelemetry.auto")
+                    .setVersion("1.0.0")
+                    .build())
             .setResource(
                 Resource.create(
                     Attributes.of(
@@ -202,7 +206,9 @@ void testExportMultipleResources() throws Exception {
             .setTotalRecordedLinks(0)
             .setTotalRecordedEvents(0)
             .setInstrumentationScopeInfo(
-                InstrumentationScopeInfo.create("io.opentelemetry.auto", "1.0.0", null))
+                InstrumentationScopeInfo.builder("io.opentelemetry.auto")
+                    .setVersion("1.0.0")
+                    .build())
             .setResource(
                 Resource.create(
                     Attributes.of(

diff --git a/.../src/test/java/io/opentelemetry/exporter/logging/otlp/OtlpJsonLoggingLogExporterTest.java b/.../src/test/java/io/opentelemetry/exporter/logging/otlp/OtlpJsonLoggingLogExporterTest.java
@@ -61,7 +61,7 @@ class OtlpJsonLoggingLogExporterTest {
       TestLogData.builder()
           .setResource(RESOURCE)
           .setInstrumentationScopeInfo(
-              InstrumentationScopeInfo.create("instrumentation2", "2", null))
+              InstrumentationScopeInfo.builder("instrumentation2").setVersion("2").build())
           .setBody("body2")
           .setSeverity(Severity.INFO)
           .setSeverityText("INFO")

diff --git a/...c/test/java/io/opentelemetry/exporter/logging/otlp/OtlpJsonLoggingMetricExporterTest.java b/...c/test/java/io/opentelemetry/exporter/logging/otlp/OtlpJsonLoggingMetricExporterTest.java
@@ -53,7 +53,7 @@ class OtlpJsonLoggingMetricExporterTest {
   private static final MetricData METRIC2 =
       ImmutableMetricData.createDoubleSum(
           RESOURCE,
-          InstrumentationScopeInfo.create("instrumentation2", "2", null),
+          InstrumentationScopeInfo.builder("instrumentation2").setVersion("2").build(),
           "metric2",
           "metric2 description",
           "s",

diff --git a/...src/test/java/io/opentelemetry/exporter/logging/otlp/OtlpJsonLoggingSpanExporterTest.java b/...src/test/java/io/opentelemetry/exporter/logging/otlp/OtlpJsonLoggingSpanExporterTest.java
@@ -86,7 +86,7 @@ class OtlpJsonLoggingSpanExporterTest {
           .setKind(SpanKind.CLIENT)
           .setResource(RESOURCE)
           .setInstrumentationScopeInfo(
-              InstrumentationScopeInfo.create("instrumentation2", "2", null))
+              InstrumentationScopeInfo.builder("instrumentation2").setVersion("2").build())
           .build();
 
   @RegisterExtension

diff --git a/...rs/logging/src/test/java/io/opentelemetry/exporter/logging/LoggingMetricExporterTest.java b/...rs/logging/src/test/java/io/opentelemetry/exporter/logging/LoggingMetricExporterTest.java
@@ -63,7 +63,7 @@ void testExport() {
     long nowEpochNanos = System.currentTimeMillis() * 1000 * 1000;
     Resource resource = Resource.create(Attributes.of(stringKey("host"), "localhost"));
     InstrumentationScopeInfo instrumentationScopeInfo =
-        InstrumentationScopeInfo.create("manualInstrumentation", "1.0", null);
+        InstrumentationScopeInfo.builder("manualInstrumentation").setVersion("1.0").build();
     exporter.export(
         Arrays.asList(
             ImmutableMetricData.createDoubleSummary(

diff --git a/...ters/logging/src/test/java/io/opentelemetry/exporter/logging/LoggingSpanExporterTest.java b/...ters/logging/src/test/java/io/opentelemetry/exporter/logging/LoggingSpanExporterTest.java
@@ -82,7 +82,8 @@ class LoggingSpanExporterTest {
           .setStatus(StatusData.error())
           .setName("testSpan2")
           .setKind(SpanKind.CLIENT)
-          .setInstrumentationScopeInfo(InstrumentationScopeInfo.create("tracer2", "1.0", null))
+          .setInstrumentationScopeInfo(
+              InstrumentationScopeInfo.builder("tracer2").setVersion("1.0").build())
           .build();
 
   @RegisterExtension

diff --git a/...ers/logging/src/test/java/io/opentelemetry/exporter/logging/SystemOutLogExporterTest.java b/...ers/logging/src/test/java/io/opentelemetry/exporter/logging/SystemOutLogExporterTest.java
@@ -53,7 +53,8 @@ void format() {
   private static LogData sampleLog(long timestamp) {
     return TestLogData.builder()
         .setResource(Resource.empty())
-        .setInstrumentationScopeInfo(InstrumentationScopeInfo.create("logTest", "1.0", null))
+        .setInstrumentationScopeInfo(
+            InstrumentationScopeInfo.builder("logTest").setVersion("1.0").build())
         .setAttributes(Attributes.of(stringKey("cheese"), "cheddar", longKey("amount"), 1L))
         .setBody("message")
         .setSeverity(Severity.ERROR3)

diff --git a/...all/src/test/java/io/opentelemetry/exporter/otlp/http/trace/OtlpHttpSpanExporterTest.java b/...all/src/test/java/io/opentelemetry/exporter/otlp/http/trace/OtlpHttpSpanExporterTest.java
@@ -347,7 +347,10 @@ private static SpanData generateFakeSpan() {
         .setTotalRecordedLinks(0)
         .setTotalRecordedEvents(0)
         .setInstrumentationScopeInfo(
-            InstrumentationScopeInfo.create("testLib", "1.0", "http://url"))
+            InstrumentationScopeInfo.builder("testLib")
+                .setVersion("1.0")
+                .setSchemaUrl("http://url")
+                .build())
         .build();
   }
 }
diff --git a/...otlp/all/src/test/java/io/opentelemetry/exporter/otlp/trace/OtlpGrpcSpanExporterTest.java b/...otlp/all/src/test/java/io/opentelemetry/exporter/otlp/trace/OtlpGrpcSpanExporterTest.java
@@ -78,7 +78,10 @@ protected SpanData generateFakeTelemetry() {
         .setTotalRecordedLinks(0)
         .setTotalRecordedEvents(0)
         .setInstrumentationScopeInfo(
-            InstrumentationScopeInfo.create("testLib", "1.0", "http://url"))
+            InstrumentationScopeInfo.builder("testLib")
+                .setVersion("1.0")
+                .setSchemaUrl("http://url")
+                .build())
         .build();
   }
 

diff --git a/...estGrpcNetty/java/io/opentelemetry/exporter/otlp/trace/OtlpGrpcNettySpanExporterTest.java b/...estGrpcNetty/java/io/opentelemetry/exporter/otlp/trace/OtlpGrpcNettySpanExporterTest.java
@@ -86,7 +86,10 @@ protected SpanData generateFakeTelemetry() {
         .setTotalRecordedLinks(0)
         .setTotalRecordedEvents(0)
         .setInstrumentationScopeInfo(
-            InstrumentationScopeInfo.create("testLib", "1.0", "http://url"))
+            InstrumentationScopeInfo.builder("testLib")
+                .setVersion("1.0")
+                .setSchemaUrl("http://url")
+                .build())
         .build();
   }
 

diff --git a/...Shaded/java/io/opentelemetry/exporter/otlp/trace/OtlpGrpcNettyShadedSpanExporterTest.java b/...Shaded/java/io/opentelemetry/exporter/otlp/trace/OtlpGrpcNettyShadedSpanExporterTest.java
@@ -74,7 +74,10 @@ protected SpanData generateFakeTelemetry() {
         .setTotalRecordedLinks(0)
         .setTotalRecordedEvents(0)
         .setInstrumentationScopeInfo(
-            InstrumentationScopeInfo.create("testLib", "1.0", "http://url"))
+            InstrumentationScopeInfo.builder("testLib")
+                .setVersion("1.0")
+                .setSchemaUrl("http://url")
+                .build())
         .build();
   }
 

diff --git a/...tGrpcOkhttp/java/io/opentelemetry/exporter/otlp/trace/OtlpGrpcOkHttpSpanExporterTest.java b/...tGrpcOkhttp/java/io/opentelemetry/exporter/otlp/trace/OtlpGrpcOkHttpSpanExporterTest.java
@@ -74,7 +74,10 @@ protected SpanData generateFakeTelemetry() {
         .setTotalRecordedLinks(0)
         .setTotalRecordedEvents(0)
         .setInstrumentationScopeInfo(
-            InstrumentationScopeInfo.create("testLib", "1.0", "http://url"))
+            InstrumentationScopeInfo.builder("testLib")
+                .setVersion("1.0")
+                .setSchemaUrl("http://url")
+                .build())
         .build();
   }
 

diff --git a/.../src/test/java/io/opentelemetry/exporter/internal/otlp/logs/LogsRequestMarshalerTest.java b/.../src/test/java/io/opentelemetry/exporter/internal/otlp/logs/LogsRequestMarshalerTest.java
@@ -103,7 +103,7 @@ void toProtoLogRecord() {
                     .setResource(
                         Resource.create(Attributes.builder().put("testKey", "testValue").build()))
                     .setInstrumentationScopeInfo(
-                        InstrumentationScopeInfo.create("instrumentation", "1", null))
+                        InstrumentationScopeInfo.builder("instrumentation").setVersion("1").build())
                     .setBody(BODY)
                     .setSeverity(Severity.INFO)
                     .setSeverityText("INFO")
@@ -137,7 +137,7 @@ void toProtoLogRecord_MinimalFields() {
                     .setResource(
                         Resource.create(Attributes.builder().put("testKey", "testValue").build()))
                     .setInstrumentationScopeInfo(
-                        InstrumentationScopeInfo.create("instrumentation", "1", null))
+                        InstrumentationScopeInfo.builder("instrumentation").setVersion("1").build())
                     .setEpoch(12345, TimeUnit.NANOSECONDS)
                     .build()));
 

diff --git a/.../logs/src/test/java/io/opentelemetry/exporter/otlp/http/logs/OtlpHttpLogExporterTest.java b/.../logs/src/test/java/io/opentelemetry/exporter/otlp/http/logs/OtlpHttpLogExporterTest.java
@@ -342,7 +342,10 @@ private static LogData generateFakeLog() {
     return TestLogData.builder()
         .setResource(Resource.getDefault())
         .setInstrumentationScopeInfo(
-            InstrumentationScopeInfo.create("testLib", "1.0", "http://url"))
+            InstrumentationScopeInfo.builder("testLib")
+                .setVersion("1.0")
+                .setSchemaUrl("http://url")
+                .build())
         .setBody("log body")
         .setAttributes(Attributes.builder().put("key", "value").build())
         .setSeverity(Severity.INFO)

diff --git a/.../otlp/logs/src/test/java/io/opentelemetry/exporter/otlp/logs/OtlpGrpcLogExporterTest.java b/.../otlp/logs/src/test/java/io/opentelemetry/exporter/otlp/logs/OtlpGrpcLogExporterTest.java
@@ -58,7 +58,8 @@ protected TelemetryExporterBuilder<LogData> exporterBuilder() {
   protected LogData generateFakeTelemetry() {
     return TestLogData.builder()
         .setResource(Resource.create(Attributes.builder().put("testKey", "testValue").build()))
-        .setInstrumentationScopeInfo(InstrumentationScopeInfo.create("instrumentation", "1", null))
+        .setInstrumentationScopeInfo(
+            InstrumentationScopeInfo.builder("instrumentation").setVersion("1").build())
         .setEpoch(Instant.now())
         .setSeverity(Severity.ERROR)
         .setSeverityText("really severe")

diff --git a/.../testGrpcNetty/java/io/opentelemetry/exporter/otlp/logs/OtlpGrpcNettyLogExporterTest.java b/.../testGrpcNetty/java/io/opentelemetry/exporter/otlp/logs/OtlpGrpcNettyLogExporterTest.java
@@ -66,7 +66,8 @@ protected TelemetryExporterBuilder<LogData> exporterBuilder() {
   protected LogData generateFakeTelemetry() {
     return TestLogData.builder()
         .setResource(Resource.create(Attributes.builder().put("testKey", "testValue").build()))
-        .setInstrumentationScopeInfo(InstrumentationScopeInfo.create("instrumentation", "1", null))
+        .setInstrumentationScopeInfo(
+            InstrumentationScopeInfo.builder("instrumentation").setVersion("1").build())
         .setEpoch(Instant.now())
         .setSeverity(Severity.ERROR)
         .setSeverityText("really severe")

diff --git a/...tyShaded/java/io/opentelemetry/exporter/otlp/logs/OtlpGrpcNettyShadedLogExporterTest.java b/...tyShaded/java/io/opentelemetry/exporter/otlp/logs/OtlpGrpcNettyShadedLogExporterTest.java
@@ -54,7 +54,8 @@ protected TelemetryExporterBuilder<LogData> exporterBuilder() {
   protected LogData generateFakeTelemetry() {
     return TestLogData.builder()
         .setResource(Resource.create(Attributes.builder().put("testKey", "testValue").build()))
-        .setInstrumentationScopeInfo(InstrumentationScopeInfo.create("instrumentation", "1", null))
+        .setInstrumentationScopeInfo(
+            InstrumentationScopeInfo.builder("instrumentation").setVersion("1").build())
         .setEpoch(Instant.now())
         .setSeverity(Severity.ERROR)
         .setSeverityText("really severe")

diff --git a/...pcOkhttp/java/io/opentelemetry/exporter/otlp/logs/OtlpGrpcNettyOkHttpLogExporterTest.java b/...pcOkhttp/java/io/opentelemetry/exporter/otlp/logs/OtlpGrpcNettyOkHttpLogExporterTest.java
@@ -54,7 +54,8 @@ protected TelemetryExporterBuilder<LogData> exporterBuilder() {
   protected LogData generateFakeTelemetry() {
     return TestLogData.builder()
         .setResource(Resource.create(Attributes.builder().put("testKey", "testValue").build()))
-        .setInstrumentationScopeInfo(InstrumentationScopeInfo.create("instrumentation", "1", null))
+        .setInstrumentationScopeInfo(
+            InstrumentationScopeInfo.builder("instrumentation").setVersion("1").build())
         .setEpoch(Instant.now())
         .setSeverity(Severity.ERROR)
         .setSeverityText("really severe")

diff --git a/...heus/src/integrationTest/java/io/opentelemetry/exporter/prometheus/MetricAdapterTest.java b/...heus/src/integrationTest/java/io/opentelemetry/exporter/prometheus/MetricAdapterTest.java
@@ -52,7 +52,7 @@ class MetricAdapterTest {
   private static final MetricData MONOTONIC_CUMULATIVE_DOUBLE_SUM =
       ImmutableMetricData.createDoubleSum(
           Resource.create(Attributes.of(stringKey("kr"), "vr")),
-          InstrumentationScopeInfo.create("full", "version", null),
+          InstrumentationScopeInfo.builder("full").setVersion("version").build(),
           "instrument.name",
           "description",
           "1",
@@ -65,7 +65,7 @@ class MetricAdapterTest {
   private static final MetricData NON_MONOTONIC_CUMULATIVE_DOUBLE_SUM =
       ImmutableMetricData.createDoubleSum(
           Resource.create(Attributes.of(stringKey("kr"), "vr")),
-          InstrumentationScopeInfo.create("full", "version", null),
+          InstrumentationScopeInfo.builder("full").setVersion("version").build(),
           "instrument.name",
           "description",
           "1",
@@ -78,7 +78,7 @@ class MetricAdapterTest {
   private static final MetricData MONOTONIC_DELTA_DOUBLE_SUM =
       ImmutableMetricData.createDoubleSum(
           Resource.create(Attributes.of(stringKey("kr"), "vr")),
-          InstrumentationScopeInfo.create("full", "version", null),
+          InstrumentationScopeInfo.builder("full").setVersion("version").build(),
           "instrument.name",
           "description",
           "1",
@@ -91,7 +91,7 @@ class MetricAdapterTest {
   private static final MetricData NON_MONOTONIC_DELTA_DOUBLE_SUM =
       ImmutableMetricData.createDoubleSum(
           Resource.create(Attributes.of(stringKey("kr"), "vr")),
-          InstrumentationScopeInfo.create("full", "version", null),
+          InstrumentationScopeInfo.builder("full").setVersion("version").build(),
           "instrument.name",
           "description",
           "1",
@@ -104,7 +104,7 @@ class MetricAdapterTest {
   private static final MetricData MONOTONIC_CUMULATIVE_LONG_SUM =
       ImmutableMetricData.createLongSum(
           Resource.create(Attributes.of(stringKey("kr"), "vr")),
-          InstrumentationScopeInfo.create("full", "version", null),
+          InstrumentationScopeInfo.builder("full").setVersion("version").build(),
           "instrument.name",
           "description",
           "1",
@@ -117,7 +117,7 @@ class MetricAdapterTest {
   private static final MetricData NON_MONOTONIC_CUMULATIVE_LONG_SUM =
       ImmutableMetricData.createLongSum(
           Resource.create(Attributes.of(stringKey("kr"), "vr")),
-          InstrumentationScopeInfo.create("full", "version", null),
+          InstrumentationScopeInfo.builder("full").setVersion("version").build(),
           "instrument.name",
           "description",
           "1",
@@ -130,7 +130,7 @@ class MetricAdapterTest {
   private static final MetricData MONOTONIC_DELTA_LONG_SUM =
       ImmutableMetricData.createLongSum(
           Resource.create(Attributes.of(stringKey("kr"), "vr")),
-          InstrumentationScopeInfo.create("full", "version", null),
+          InstrumentationScopeInfo.builder("full").setVersion("version").build(),
           "instrument.name",
           "description",
           "1",
@@ -143,7 +143,7 @@ class MetricAdapterTest {
   private static final MetricData NON_MONOTONIC_DELTA_LONG_SUM =
       ImmutableMetricData.createLongSum(
           Resource.create(Attributes.of(stringKey("kr"), "vr")),
-          InstrumentationScopeInfo.create("full", "version", null),
+          InstrumentationScopeInfo.builder("full").setVersion("version").build(),
           "instrument.name",
           "description",
           "1",
@@ -157,7 +157,7 @@ class MetricAdapterTest {
   private static final MetricData DOUBLE_GAUGE =
       ImmutableMetricData.createDoubleGauge(
           Resource.create(Attributes.of(stringKey("kr"), "vr")),
-          InstrumentationScopeInfo.create("full", "version", null),
+          InstrumentationScopeInfo.builder("full").setVersion("version").build(),
           "instrument.name",
           "description",
           "1",
@@ -168,7 +168,7 @@ class MetricAdapterTest {
   private static final MetricData LONG_GAUGE =
       ImmutableMetricData.createLongGauge(
           Resource.create(Attributes.of(stringKey("kr"), "vr")),
-          InstrumentationScopeInfo.create("full", "version", null),
+          InstrumentationScopeInfo.builder("full").setVersion("version").build(),
           "instrument.name",
           "description",
           "1",
@@ -179,7 +179,7 @@ class MetricAdapterTest {
   private static final MetricData SUMMARY =
       ImmutableMetricData.createDoubleSummary(
           Resource.create(Attributes.of(stringKey("kr"), "vr")),
-          InstrumentationScopeInfo.create("full", "version", null),
+          InstrumentationScopeInfo.builder("full").setVersion("version").build(),
           "instrument.name",
           "description",
           "1",
@@ -197,7 +197,7 @@ class MetricAdapterTest {
   private static final MetricData HISTOGRAM =
       ImmutableMetricData.createDoubleHistogram(
           Resource.create(Attributes.of(stringKey("kr"), "vr")),
-          InstrumentationScopeInfo.create("full", "version", null),
+          InstrumentationScopeInfo.builder("full").setVersion("version").build(),
           "instrument.name",
           "description",
           "1",

diff --git a/...rc/integrationTest/java/io/opentelemetry/exporter/prometheus/PrometheusCollectorTest.java b/...rc/integrationTest/java/io/opentelemetry/exporter/prometheus/PrometheusCollectorTest.java
@@ -79,7 +79,7 @@ private static ImmutableList<MetricData> generateTestData() {
     return ImmutableList.of(
         ImmutableMetricData.createLongSum(
             Resource.create(Attributes.of(stringKey("kr"), "vr")),
-            InstrumentationScopeInfo.create("grpc", "version", null),
+            InstrumentationScopeInfo.builder("grpc").setVersion("version").build(),
             "grpc.name",
             "long_description",
             "1",
@@ -94,7 +94,7 @@ private static ImmutableList<MetricData> generateTestData() {
                         5)))),
         ImmutableMetricData.createDoubleSum(
             Resource.create(Attributes.of(stringKey("kr"), "vr")),
-            InstrumentationScopeInfo.create("http", "version", null),
+            InstrumentationScopeInfo.builder("http").setVersion("version").build(),
             "http.name",
             "double_description",
             "1",