Metrics SDK does not comply with Service/Telemetry SDK semantic conventions

[pichlermarc]

What happened?

Steps to Reproduce

Setup Metrics SDK and export data anywhere.

Expected Result

Similar to the Trace SDK, I expected that Resource includes the attributes listed in the Service Semantic Conventions and Telemetry SDK Semantic Conventions, which state that:

• service.name is REQUIRED
• telemetry.sdk.name is RECOMMENDED
• telemetry.sdk.language is RECOMMENDED
• telemetry.sdk.versionis RECOMMENDED

Actual Result

Resource is empty.

OpenTelemetry Setup Code

'use strict';

const { OTLPMetricExporter } = require('@opentelemetry/exporter-metrics-otlp-grpc');
const { MeterProvider, PeriodicExportingMetricReader } = require('@opentelemetry/sdk-metrics');

const metricExporter = new OTLPMetricExporter({});
const meterProvider = new MeterProvider({});

meterProvider.addMetricReader(new PeriodicExportingMetricReader({
  exporter: metricExporter,
  exportIntervalMillis: 1000,
}));

const meter = meterProvider.getMeter('my-meter');
const counter = meter.createCounter('counter');

setInterval(() => {
  counter.add(1);
}, 1000);


// Collector config for logging the metrics:
receivers:
  otlp:
    protocols:
      grpc:

exporters:
  logging:
    loglevel: "debug"

service:
  pipelines:
    metrics:
      receivers: [otlp]
      exporters: [logging]

package.json

{
  "name": "semconv-bug-repro",
  "scripts": {
    "start:metrics": "node metrics.js"
  },
  "author": "OpenTelemetry Authors",
  "license": "Apache-2.0",
  "dependencies": {
    "@opentelemetry/api": "^1.3.0",
    "@opentelemetry/exporter-metrics-otlp-grpc": "0.34.0",
    "@opentelemetry/sdk-metrics": "1.8.0"
  }
}

Relevant log output

Resource SchemaURL: 
 ScopeMetrics #0
 ScopeMetrics SchemaURL: 
 InstrumentationScope my-meter 
 Metric #0
 Descriptor:
      -> Name: counter
      -> Description: 
      -> Unit: 
      -> DataType: Sum
      -> IsMonotonic: true
      -> AggregationTemporality: AGGREGATION_TEMPORALITY_CUMULATIVE
 NumberDataPoints #0
 StartTimestamp: 2022-11-14 10:31:06.026851072 +0000 UTC
 Timestamp: 2022-11-14 10:37:36.361387264 +0000 UTC
 Value: 390.000000

[pichlermarc]

I'm working on a fix for this 🙂

[dyladan]

Please add priority labels when you can