Accept review by aluzzardi

Signed-off-by: Alvise <vitalvise@gmail.com>

cmd/dagger/logger/tracer.go

@@ -14,7 +14,7 @@ import (
 )
 
 type tracer struct {
-	TraceProvider *tracesdk.TracerProvider
+	provider *tracesdk.TracerProvider
 }
 
 func InitTracing() io.Closer {
@@ -33,7 +33,7 @@ func InitTracing() io.Closer {
 	otel.SetTracerProvider(tp)
 
 	tracer := tracer{
-		TraceProvider: tp,
+		provider: tp,
 	}
 
 	return tracer
@@ -51,34 +51,20 @@ func tracerProvider(url string) (*tracesdk.TracerProvider, error) {
 	}
 	tp := tracesdk.NewTracerProvider(
 		// Always be sure to batch in production.
-		tracesdk.WithBatcher(exp),
+		tracesdk.WithBatcher(exp, tracesdk.WithMaxExportBatchSize(1)),
 		// Record information about this application in an Resource.
 		tracesdk.WithResource(resource.NewWithAttributes(
 			semconv.SchemaURL,
 			semconv.ServiceNameKey.String("dagger"),
-			// attribute.String("environment", "dagger"),
-			// attribute.Int64("ID", 1),
 		)),
 	)
 	return tp, nil
 }
 
 func (t tracer) Close() error {
-	ctx, cancel := context.WithCancel(context.Background())
+	ctx, cancel := context.WithTimeout(context.Background(), time.Second*5)
 	defer cancel()
-
-	var closeError error
-	// Cleanly shutdown and flush telemetry when the application exits.
-	defer func(ctx context.Context) {
-		// Do not make the application hang when it is shutdown.
-		ctx, cancel = context.WithTimeout(ctx, time.Second*5)
-		defer cancel()
-		if err := t.TraceProvider.Shutdown(ctx); err != nil {
-			closeError = err
-		}
-	}(ctx)
-
-	return closeError
+	return t.provider.Shutdown(ctx)
 }
 
 type nopCloser struct {