Added val with all metric qualifications (#20418)

Before this change, we had to refer to each qualification individually in order to refer to all of them. Now, there is a list of qualifiations that can be used instead.
digital-asset · Dec 6, 2024 · bb8ba5c · bb8ba5c
1 parent e43ba71
commit bb8ba5c
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 49 deletions.
diff --git a/sdk/canton/community/common/src/main/scala/com/digitalasset/canton/metrics/MetricDoc.scala b/sdk/canton/community/common/src/main/scala/com/digitalasset/canton/metrics/MetricDoc.scala
@@ -3,13 +3,11 @@
 
 package com.digitalasset.canton.metrics
 
-import com.daml.metrics.api.MetricDoc.Tag
 import com.daml.metrics.api.MetricQualification
 
 object MetricDoc {
 
   final case class Item(
-      tag: Tag,
       name: String,
       metricType: String,
       qualification: MetricQualification,

diff --git a/sdk/observability/metrics/src/main/scala/com/daml/metrics/api/MetricDoc.scala b/sdk/observability/metrics/src/main/scala/com/daml/metrics/api/MetricDoc.scala
diff --git a/sdk/observability/metrics/src/main/scala/com/daml/metrics/api/MetricQualification.scala b/sdk/observability/metrics/src/main/scala/com/daml/metrics/api/MetricQualification.scala
@@ -7,6 +7,9 @@ package com.daml.metrics.api
 sealed trait MetricQualification
 object MetricQualification {
 
+  /** All possible metric qualifications */
+  lazy val All: Seq[MetricQualification] = Seq(Latency, Traffic, Errors, Saturation, Debug)
+
   /** Any metric measuring processing time of some sort */
   case object Latency extends MetricQualification