Add unit test codifying that view names need to conform to metric nam…

exporters/otlp/all/build.gradle.kts

@@ -10,8 +10,8 @@ apply<io.opentelemetry.gradle.OtelVersionClassPlugin>()
 description = "OpenTelemetry Protocol (OTLP) Exporters"
 otelJava.moduleName.set("io.opentelemetry.exporter.otlp")
 otelJava.osgiOptionalPackages.set(listOf("io.opentelemetry.api.incubator.config"))
-// io.grpc is not an OSGi bundle and has no package versioning; must use unversioned optional.
-otelJava.osgiUnversionedOptionalPackages.set(listOf("io.grpc"))
+// io.grpc and org.jspecify.annotations are not OSGi bundles; must use unversioned optional.
+otelJava.osgiUnversionedOptionalPackages.set(listOf("io.grpc", "org.jspecify.annotations"))
 base.archivesName.set("opentelemetry-exporter-otlp")
 
 dependencies {

sdk/metrics/src/test/java/io/opentelemetry/sdk/metrics/IdentityTest.java

@@ -8,6 +8,7 @@
 import static io.opentelemetry.sdk.testing.assertj.OpenTelemetryAssertions.assertThat;
 
 import io.github.netmikey.logunit.api.LogCapturer;
+import io.opentelemetry.api.metrics.LongCounter;
 import io.opentelemetry.internal.testing.slf4j.SuppressLogger;
 import io.opentelemetry.sdk.common.InstrumentationScopeInfo;
 import io.opentelemetry.sdk.metrics.internal.state.MetricStorageRegistry;
@@ -28,6 +29,8 @@ class IdentityTest {
   @RegisterExtension
   LogCapturer viewRegistryLogs = LogCapturer.create().captureForType(ViewRegistry.class);
 
+  @RegisterExtension LogCapturer sdkMeterLogs = LogCapturer.create().captureForType(SdkMeter.class);
+
   private InMemoryMetricReader reader;
   private SdkMeterProviderBuilder builder;
 
@@ -1092,6 +1095,55 @@ void sameMeterDifferentInstrumentIncompatibleViewAggregation() {
         .hasSize(1);
   }
 
+  @Test
+  void viewNameNotValidated() {
+    // View names are not validated against the instrument name pattern. When a view sets a name
+    // that would be rejected if used directly as an instrument name (e.g. starts with a digit),
+    // the SDK accepts it and produces a metric with the invalid name without logging a warning.
+    SdkMeterProvider meterProvider =
+        builder
+            .registerView(
+                InstrumentSelector.builder().setType(InstrumentType.COUNTER).build(),
+                View.builder().setName("1invalid-name").build())
+            .build();
+
+    LongCounter counter = meterProvider.get("meter1").counterBuilder("counter1").build();
+    counter.add(10);
+
+    assertThat(counter.isEnabled()).isTrue();
+    assertThat(reader.collectAllMetrics())
+        .satisfiesExactlyInAnyOrder(
+            metricData ->
+                assertThat(metricData)
+                    .hasInstrumentationScope(forMeter("meter1"))
+                    .hasName("1invalid-name")
+                    .hasLongSumSatisfying(
+                        sum -> sum.hasPointsSatisfying(point -> point.hasValue(10))));
+
+    assertThat(metricStorageRegistryLogs.getEvents()).hasSize(0);
+    assertThat(viewRegistryLogs.getEvents()).hasSize(0);
+  }
+
+  @Test
+  @SuppressLogger(SdkMeter.class)
+  void instrumentWithInvalidName() {
+    // When an instrument is initialized with an invalid name, a noop instrument is returned
+    // and a warning is logged. No metric data is produced.
+    SdkMeterProvider meterProvider = builder.build();
+
+    LongCounter counter = meterProvider.get("meter1").counterBuilder("1invalid-name").build();
+    counter.add(10);
+
+    assertThat(counter.isEnabled()).isFalse();
+    assertThat(reader.collectAllMetrics()).isEmpty();
+    assertThat(sdkMeterLogs.getEvents())
+        .allSatisfy(
+            logEvent ->
+                assertThat(logEvent.getMessage())
+                    .contains("Instrument name \"1invalid-name\" is invalid"))
+        .hasSize(1);
+  }
+
   private static InstrumentationScopeInfo forMeter(String meterName) {
     return InstrumentationScopeInfo.create(meterName);
   }