From ce570ab8e89009479dbfbb9dddfe6a02fd3a0c79 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Robert=20Paj=C4=85k?= Date: Thu, 25 Jan 2024 20:22:31 +0100 Subject: [PATCH] [Go] Getting started: Do not use semconv --- .../en/docs/languages/go/getting-started.md | 34 ++++--------------- 1 file changed, 7 insertions(+), 27 deletions(-) diff --git a/content/en/docs/languages/go/getting-started.md b/content/en/docs/languages/go/getting-started.md index e71b38ec154f..c6488eacdb77 100644 --- a/content/en/docs/languages/go/getting-started.md +++ b/content/en/docs/languages/go/getting-started.md @@ -134,14 +134,12 @@ import ( "go.opentelemetry.io/otel/exporters/stdout/stdouttrace" "go.opentelemetry.io/otel/propagation" "go.opentelemetry.io/otel/sdk/metric" - "go.opentelemetry.io/otel/sdk/resource" "go.opentelemetry.io/otel/sdk/trace" - semconv "go.opentelemetry.io/otel/semconv/v1.21.0" ) // setupOTelSDK bootstraps the OpenTelemetry pipeline. // If it does not return an error, make sure to call shutdown for proper cleanup. -func setupOTelSDK(ctx context.Context, serviceName, serviceVersion string) (shutdown func(context.Context) error, err error) { +func setupOTelSDK(ctx context.Context) (shutdown func(context.Context) error, err error) { var shutdownFuncs []func(context.Context) error // shutdown calls cleanup functions registered via shutdownFuncs. @@ -161,19 +159,12 @@ func setupOTelSDK(ctx context.Context, serviceName, serviceVersion string) (shut err = errors.Join(inErr, shutdown(ctx)) } - // Set up resource. - res, err := newResource(serviceName, serviceVersion) - if err != nil { - handleErr(err) - return - } - // Set up propagator. prop := newPropagator() otel.SetTextMapPropagator(prop) // Set up trace provider. - tracerProvider, err := newTraceProvider(res) + tracerProvider, err := newTraceProvider() if err != nil { handleErr(err) return @@ -182,7 +173,7 @@ func setupOTelSDK(ctx context.Context, serviceName, serviceVersion string) (shut otel.SetTracerProvider(tracerProvider) // Set up meter provider. - meterProvider, err := newMeterProvider(res) + meterProvider, err := newMeterProvider() if err != nil { handleErr(err) return @@ -193,14 +184,6 @@ func setupOTelSDK(ctx context.Context, serviceName, serviceVersion string) (shut return } -func newResource(serviceName, serviceVersion string) (*resource.Resource, error) { - return resource.Merge(resource.Default(), - resource.NewWithAttributes(semconv.SchemaURL, - semconv.ServiceName(serviceName), - semconv.ServiceVersion(serviceVersion), - )) -} - func newPropagator() propagation.TextMapPropagator { return propagation.NewCompositeTextMapPropagator( propagation.TraceContext{}, @@ -208,7 +191,7 @@ func newPropagator() propagation.TextMapPropagator { ) } -func newTraceProvider(res *resource.Resource) (*trace.TracerProvider, error) { +func newTraceProvider() (*trace.TracerProvider, error) { traceExporter, err := stdouttrace.New( stdouttrace.WithPrettyPrint()) if err != nil { @@ -219,19 +202,17 @@ func newTraceProvider(res *resource.Resource) (*trace.TracerProvider, error) { trace.WithBatcher(traceExporter, // Default is 5s. Set to 1s for demonstrative purposes. trace.WithBatchTimeout(time.Second)), - trace.WithResource(res), ) return traceProvider, nil } -func newMeterProvider(res *resource.Resource) (*metric.MeterProvider, error) { +func newMeterProvider() (*metric.MeterProvider, error) { metricExporter, err := stdoutmetric.New() if err != nil { return nil, err } meterProvider := metric.NewMeterProvider( - metric.WithResource(res), metric.WithReader(metric.NewPeriodicReader(metricExporter, // Default is 1m. Set to 3s for demonstrative purposes. metric.WithInterval(3*time.Second))), @@ -279,9 +260,7 @@ func run() (err error) { defer stop() // Set up OpenTelemetry. - serviceName := "dice" - serviceVersion := "0.1.0" - otelShutdown, err := setupOTelSDK(ctx, serviceName, serviceVersion) + otelShutdown, err := setupOTelSDK(ctx) if err != nil { return } @@ -406,6 +385,7 @@ Build and run the application with the following command: ```sh go mod tidy +export OTEL_RESOURCE_ATTRIBUTES="service.name=dice,service.version=0.1.0" go run . ```