valueMax reporting wrong number

[razum90]

Expected behavior

I would expect valueMax to have a value less than valueSum.

Actual behavior

valueMax is greater than valueSum.

To Reproduce

This is my method that is called to gather metrics using micrometer:

@RequiredArgsConstructor
@Slf4j
public class TimeRecorder {
    private final static String METHOD = "method";
    private final static String CLASS = "class";
    private final MeterRegistry registry;

    public <T> T recorded(Supplier<T> supplier) {
        StackTraceElement[] stackTrace = Thread.currentThread().getStackTrace();

        if (stackTrace.length >= 3) {
            StackTraceElement stackTraceElement = stackTrace[2];

            return registry.timer("method.timing",
                            List.of(
                                    Tag.of(CLASS, stackTraceElement.getClassName()),
                                    Tag.of(METHOD, stackTraceElement.getMethodName())
                            ))
                    .record(supplier);
        } else {
            log.error("unable to determine callee, skipping metrics");
            return supplier.get();
        }
    }
}

System information

Please provide the following information:

• SDK Version: applicationinsights-agent-3.4.6.jar
• OS type and version: eclipse-temurin:17-jdk-alpine
• Application Server type and version (if applicable):
• Using spring-boot? Yes (version 2.7.1)
• Additional relevant libraries (with version, if applicable): Micrometer 1.9.1

Logs

Turn on SDK logs and attach/paste them to the issue. If using an application server, also attach any relevant server logs.

Be sure to remove any private information from the logs before posting!

Screenshots

If applicable, add screenshots to help explain your problem.

[trask]

hi @razum90, thanks for reporting this! I'm able to repro this locally, it looks like the valueMax is the max since JVM start instead of the max for the given interval. We will investigate further, thanks.

[razum90]

Thanks @trask!

[trask]

hi @razum90, this looks like a micrometer bug: micrometer-metrics/micrometer#3298

Application Insights Java also supports OpenTelemetry Metrics, which I confirmed doesn't have this issue:

    private static void micrometer() throws InterruptedException {
        Timer timer = Metrics.timer("test");
        timer.record(Duration.ofMillis(100));
        MINUTES.sleep(1);
        timer.record(Duration.ofMillis(10));
        MINUTES.sleep(1);
    }

    private static void opentelemetry() throws InterruptedException {
        LongHistogram timer = GlobalOpenTelemetry.getMeter("test").histogramBuilder("test").ofLongs().build();
        timer.record(100);
        MINUTES.sleep(1);
        timer.record(10);
        MINUTES.sleep(1);
    }

(unfortunately OpenTelemetry Metrics doesn't have a dedicated Timer yet)

This issue has been automatically marked as stale because it has been marked as requiring author feedback but has not had any activity for 7 days. It will be closed if no further activity occurs within 7 days of this comment.

[razum90]

Hi @trask, oh okay. Will have to follow up that issue then. Thanks for looking into it.