build(telemetry): update otelconf dependency to v0.19 (#25719)

Author: aaronc

go.mod

@@ -64,14 +64,8 @@ require (
 	go.opentelemetry.io/contrib/propagators/b3 v1.39.0
 	go.opentelemetry.io/contrib/propagators/jaeger v1.39.0
 	go.opentelemetry.io/otel v1.39.0
-	go.opentelemetry.io/otel/exporters/stdout/stdoutlog v0.15.0
-	go.opentelemetry.io/otel/exporters/stdout/stdoutmetric v1.39.0
-	go.opentelemetry.io/otel/exporters/stdout/stdouttrace v1.39.0
 	go.opentelemetry.io/otel/log v0.15.0
 	go.opentelemetry.io/otel/metric v1.39.0
-	go.opentelemetry.io/otel/sdk v1.39.0
-	go.opentelemetry.io/otel/sdk/log v0.15.0
-	go.opentelemetry.io/otel/sdk/metric v1.39.0
 	go.opentelemetry.io/otel/trace v1.39.0
 	go.uber.org/mock v0.6.0
 	go.yaml.in/yaml/v3 v3.0.4
@@ -242,6 +236,12 @@ require (
 	go.opentelemetry.io/otel/exporters/otlp/otlptrace/otlptracegrpc v1.39.0 // indirect
 	go.opentelemetry.io/otel/exporters/otlp/otlptrace/otlptracehttp v1.39.0 // indirect
 	go.opentelemetry.io/otel/exporters/prometheus v0.61.0 // indirect
+	go.opentelemetry.io/otel/exporters/stdout/stdoutlog v0.15.0 // indirect
+	go.opentelemetry.io/otel/exporters/stdout/stdoutmetric v1.39.0 // indirect
+	go.opentelemetry.io/otel/exporters/stdout/stdouttrace v1.39.0 // indirect
+	go.opentelemetry.io/otel/sdk v1.39.0 // indirect
+	go.opentelemetry.io/otel/sdk/log v0.15.0 // indirect
+	go.opentelemetry.io/otel/sdk/metric v1.39.0 // indirect
 	go.opentelemetry.io/proto/otlp v1.9.0 // indirect
 	go.uber.org/multierr v1.11.0 // indirect
 	go.uber.org/zap v1.27.0 // indirect

telemetry/README.md

@@ -15,6 +15,7 @@ This allows meters and traces to use direct references to the underlying instrum
 Create a basic OpenTelemetry configuration file which will send data to the local instance of Grafana LGTM:
 
 ```yaml
+file_format: "1.0-rc.3"
 resource:
   attributes:
     - name: service.name
@@ -24,15 +25,14 @@ tracer_provider:
   processors:
     - batch: # NOTE: you should use batch in production!
         exporter:
-          otlp:
-            protocol: grpc
+          otlp_grpc:
             endpoint: http://localhost:4317
 
 meter_provider:
   readers:
     - pull:
         exporter:
-          prometheus: # pushes directly to prometheus backend. 
+          prometheus/development: # pushes directly to prometheus backend. 
             host: 0.0.0.0
             port: 9464
             # optional: include resource attributes as constant labels
@@ -51,23 +51,19 @@ logger_provider:
   processors:
     - batch:
         exporter:
-          otlp:
-            protocol: grpc
+          otlp_grpc:
             endpoint: http://localhost:4317
 
 
 cosmos_extra:
-  trace_file: ""
-  metrics_file: ""
-  metrics_file_interval: ""
-  logs_file: ""
   instrument_host: true
   instrument_runtime: true
   propagators:
     - tracecontext
 ```
 
-For a full list of configurable options see: https://github.com/open-telemetry/opentelemetry-configuration/blob/main/examples/kitchen-sink.yaml
+For a full list of configurable options see: https://github.com/open-telemetry/opentelemetry-configuration/blob/main/examples/kitchen-sink.yaml.
+NOTE: the go implementation may not support all options, so check the go [otelconf](https://pkg.go.dev/go.opentelemetry.io/contrib/otelconf) documentation carefully to see what is actually supported.
 
 3. set the `OTEL_EXPERIMENTAL_CONFIG_FILE` environment variable to the path of the configuration file:
    `export OTEL_EXPERIMENTAL_CONFIG_FILE=path/to/config.yaml`

telemetry/config.go

@@ -4,24 +4,17 @@ import (
 	"context"
 	"fmt"
 	"os"
-	"time"
 
 	"go.opentelemetry.io/contrib/instrumentation/host"
 	"go.opentelemetry.io/contrib/instrumentation/runtime"
-	otelconf "go.opentelemetry.io/contrib/otelconf/v0.3.0"
+	"go.opentelemetry.io/contrib/otelconf"
 	"go.opentelemetry.io/contrib/propagators/b3"
 	"go.opentelemetry.io/contrib/propagators/jaeger"
 	"go.opentelemetry.io/otel"
-	"go.opentelemetry.io/otel/exporters/stdout/stdoutlog"
-	"go.opentelemetry.io/otel/exporters/stdout/stdoutmetric"
-	"go.opentelemetry.io/otel/exporters/stdout/stdouttrace"
 	logglobal "go.opentelemetry.io/otel/log/global"
 	lognoop "go.opentelemetry.io/otel/log/noop"
 	metricnoop "go.opentelemetry.io/otel/metric/noop"
 	"go.opentelemetry.io/otel/propagation"
-	logsdk "go.opentelemetry.io/otel/sdk/log"
-	metricsdk "go.opentelemetry.io/otel/sdk/metric"
-	tracesdk "go.opentelemetry.io/otel/sdk/trace"
 	tracenoop "go.opentelemetry.io/otel/trace/noop"
 	"go.yaml.in/yaml/v3"
 )
@@ -93,83 +86,6 @@ func InitializeOpenTelemetry(filePath string) error {
 	if err == nil {
 		if extraCfg.CosmosExtra != nil {
 			extra := *extraCfg.CosmosExtra
-			if extra.TraceFile != "" {
-				traceFile, err := os.OpenFile(extra.TraceFile, os.O_CREATE|os.O_WRONLY|os.O_APPEND, 0o644)
-				if err != nil {
-					return fmt.Errorf("failed to open trace file: %w", err)
-				}
-				shutdownFuncs = append(shutdownFuncs, func(ctx context.Context) error {
-					if err := traceFile.Close(); err != nil {
-						return fmt.Errorf("failed to close trace file: %w", err)
-					}
-					return nil
-				})
-				exporter, err := stdouttrace.New(
-					stdouttrace.WithWriter(traceFile),
-					// stdouttrace.WithPrettyPrint(),
-				)
-				if err != nil {
-					return fmt.Errorf("failed to create stdout trace exporter: %w", err)
-				}
-				opts = append(opts, otelconf.WithTracerProviderOptions(
-					tracesdk.WithBatcher(exporter),
-				))
-			}
-			if extra.MetricsFile != "" {
-				metricsFile, err := os.OpenFile(extra.MetricsFile, os.O_CREATE|os.O_WRONLY|os.O_APPEND, 0o644)
-				if err != nil {
-					return fmt.Errorf("failed to open metrics file: %w", err)
-				}
-				shutdownFuncs = append(shutdownFuncs, func(ctx context.Context) error {
-					if err := metricsFile.Close(); err != nil {
-						return fmt.Errorf("failed to close metrics file: %w", err)
-					}
-					return nil
-				})
-				exporter, err := stdoutmetric.New(
-					stdoutmetric.WithWriter(metricsFile),
-					// stdoutmetric.WithPrettyPrint(),
-				)
-				if err != nil {
-					return fmt.Errorf("failed to create stdout metric exporter: %w", err)
-				}
-
-				// Configure periodic reader with custom interval if specified
-				readerOpts := []metricsdk.PeriodicReaderOption{}
-				if extra.MetricsFileInterval != "" {
-					interval, err := time.ParseDuration(extra.MetricsFileInterval)
-					if err != nil {
-						return fmt.Errorf("failed to parse metrics_file_interval: %w", err)
-					}
-					readerOpts = append(readerOpts, metricsdk.WithInterval(interval))
-				}
-
-				opts = append(opts, otelconf.WithMeterProviderOptions(
-					metricsdk.WithReader(metricsdk.NewPeriodicReader(exporter, readerOpts...)),
-				))
-			}
-			if extra.LogsFile != "" {
-				logsFile, err := os.OpenFile(extra.LogsFile, os.O_CREATE|os.O_WRONLY|os.O_APPEND, 0o644)
-				if err != nil {
-					return fmt.Errorf("failed to open logs file: %w", err)
-				}
-				shutdownFuncs = append(shutdownFuncs, func(ctx context.Context) error {
-					if err := logsFile.Close(); err != nil {
-						return fmt.Errorf("failed to close logs file: %w", err)
-					}
-					return nil
-				})
-				exporter, err := stdoutlog.New(
-					stdoutlog.WithWriter(logsFile),
-					// stdoutlog.WithPrettyPrint(),
-				)
-				if err != nil {
-					return fmt.Errorf("failed to create stdout log exporter: %w", err)
-				}
-				opts = append(opts, otelconf.WithLoggerProviderOptions(
-					logsdk.WithProcessor(logsdk.NewBatchProcessor(exporter)),
-				))
-			}
 			if extra.InstrumentHost {
 				fmt.Println("Initializing host instrumentation")
 				if err := host.Start(); err != nil {