refactor: removed hardcoded metric export interval and use otel default (#1621)

beeme1mr · web-flow · commit 81c66ebf2b82 · 2025-05-06T23:29:06.000-04:00
Signed-off-by: Michael Beemer &lt;michael.beemer@dynatrace.com&gt;
diff --git a/core/pkg/telemetry/builder.go b/core/pkg/telemetry/builder.go
@@ -30,7 +30,6 @@ import (
 
 const (
 	metricsExporterOtel = "otel"
-	exportInterval      = 2 * time.Second
 )
 
 type CollectorConfig struct {
@@ -196,7 +195,7 @@ func buildMetricReader(ctx context.Context, cfg Config) (metric.Reader, error) {
 		return nil, fmt.Errorf("error creating otel metric exporter: %w", err)
 	}
 
-	return metric.NewPeriodicReader(otelExporter, metric.WithInterval(exportInterval)), nil
+	return metric.NewPeriodicReader(otelExporter), nil
 }
 
 // buildOtlpExporter is a helper to build grpc backed otlp trace exporter

Original file line number	Diff line number	Diff line change
`@@ -30,7 +30,6 @@ import (`
`30`	`30`
`31`	`31`	`const (`
`32`	`32`	`metricsExporterOtel = "otel"`
`33`		`- exportInterval = 2 * time.Second`
`34`	`33`	`)`
`35`	`34`
`36`	`35`	`type CollectorConfig struct {`
`@@ -196,7 +195,7 @@ func buildMetricReader(ctx context.Context, cfg Config) (metric.Reader, error) {`
`196`	`195`	`return nil, fmt.Errorf("error creating otel metric exporter: %w", err)`
`197`	`196`	`}`
`198`	`197`
`199`		`- return metric.NewPeriodicReader(otelExporter, metric.WithInterval(exportInterval)), nil`
	`198`	`+ return metric.NewPeriodicReader(otelExporter), nil`
`200`	`199`	`}`
`201`	`200`
`202`	`201`	`// buildOtlpExporter is a helper to build grpc backed otlp trace exporter`