New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fix time unit for process_cpu_seconds_total #881 #882

Merged

fstab merged 1 commit into main from process_cpu_seconds_total

Oct 22, 2023

Member

fstab commented Oct 22, 2023

Fixes the bug reported in #881:

In io.prometheus.metrics.instrumentation.jvm.ProcessMetrics, the value returned from callLongGetter("getProcessCpuTime", osBean) is in nanoseconds, but is scaled with Unit.millisToSeconds(processCpuTime)


          Fix time unit for process_cpu_seconds_total #881

ca75666

Signed-off-by: Fabian Stäber <fabian@fstab.de>

fstab merged commit 6843727 into main

fstab deleted the process_cpu_seconds_total branch

October 22, 2023 09:02

fstab mentioned this pull request

process_cpu_seconds_total should be scaled from nano's, not milli's #881

Closed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet