Rename jvm.cgroup.memory.usageMb to usage for consistency

rob-bygrave · rob-bygrave · commit 3b86e0c36c0d · 2025-04-15T13:08:27.000+12:00
All the other memory metrics don't have the Mb units as a suffix to the metric name, so this is more consistent
diff --git a/metrics/src/main/java/io/avaje/metrics/core/JvmCGroupMemory.java b/metrics/src/main/java/io/avaje/metrics/core/JvmCGroupMemory.java
@@ -65,7 +65,7 @@ long pctUsage() {
   }
 
   GaugeLong usage(MemSource source, boolean reportChangesOnly) {
-    return DGaugeLong.of("jvm.cgroup.memory.usageMb", source::usageMb, reportChangesOnly);
+    return DGaugeLong.of("jvm.cgroup.memory.usage", source::usageMb, reportChangesOnly);
   }
 
   GaugeLong pctUsage(MemSource source, boolean reportChangesOnly) {

Original file line number	Diff line number	Diff line change
`@@ -65,7 +65,7 @@ long pctUsage() {`
`65`	`65`	`}`
`66`	`66`
`67`	`67`	`GaugeLong usage(MemSource source, boolean reportChangesOnly) {`
`68`		`- return DGaugeLong.of("jvm.cgroup.memory.usageMb", source::usageMb, reportChangesOnly);`
	`68`	`+ return DGaugeLong.of("jvm.cgroup.memory.usage", source::usageMb, reportChangesOnly);`
`69`	`69`	`}`
`70`	`70`
`71`	`71`	`GaugeLong pctUsage(MemSource source, boolean reportChangesOnly) {`