From 5d9efb1f20cfcd6bff54cd1fc2db05b8c0466c30 Mon Sep 17 00:00:00 2001 From: Gregor Zeitlinger Date: Tue, 13 Feb 2024 17:31:12 +0100 Subject: [PATCH] use duration parser in jmx metrics (#10532) --- .../javaagent/jmx/JmxMetricInsightInstaller.java | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/instrumentation/jmx-metrics/javaagent/src/main/java/io/opentelemetry/instrumentation/javaagent/jmx/JmxMetricInsightInstaller.java b/instrumentation/jmx-metrics/javaagent/src/main/java/io/opentelemetry/instrumentation/javaagent/jmx/JmxMetricInsightInstaller.java index 69b3c09c90..8dbe6c37bd 100644 --- a/instrumentation/jmx-metrics/javaagent/src/main/java/io/opentelemetry/instrumentation/javaagent/jmx/JmxMetricInsightInstaller.java +++ b/instrumentation/jmx-metrics/javaagent/src/main/java/io/opentelemetry/instrumentation/javaagent/jmx/JmxMetricInsightInstaller.java @@ -20,6 +20,7 @@ import io.opentelemetry.sdk.autoconfigure.spi.ConfigProperties; import java.io.InputStream; import java.nio.file.Files; import java.nio.file.Paths; +import java.time.Duration; import java.util.List; /** An {@link AgentListener} that enables JMX metrics during agent startup. */ @@ -46,8 +47,9 @@ public class JmxMetricInsightInstaller implements AgentListener { // If discovery delay has not been configured, have a peek at the metric export interval. // It makes sense for both of these values to be similar. - long exportInterval = configProperties.getLong("otel.metric.export.interval", 60000); - return exportInterval; + return configProperties + .getDuration("otel.metric.export.interval", Duration.ofMinutes(1)) + .toMillis(); } private static String resourceFor(String platform) {