ref: Port both failed conferences stats to Metrics. (#1930)

* ref: Port both failed conferences stats to Metrics. Signed-off-by: Alexandre Fonseca <[email protected]> * Rename metric variables and update descriptions. * ref: Rename "failed conferences" metric variables. * doc: Update comments and metric descriptions. Signed-off-by: Alexandre Fonseca <[email protected]>
jitsi · Aug 5, 2022 · 915593c · 915593c
1 parent 0a2decd
commit 915593c
Show file tree

Hide file tree

Showing 3 changed files with 12 additions and 12 deletions.
diff --git a/jvb/src/main/java/org/jitsi/videobridge/Conference.java b/jvb/src/main/java/org/jitsi/videobridge/Conference.java
@@ -660,12 +660,12 @@ private void updateStatisticsOnExpire()
 
         if (hasPartiallyFailed)
         {
-            videobridgeStatistics.totalPartiallyFailedConferences.incrementAndGet();
+            videobridgeStatistics.partiallyFailedConferences.incAndGet();
         }
 
         if (hasFailed)
         {
-            videobridgeStatistics.totalFailedConferences.incrementAndGet();
+            videobridgeStatistics.failedConferences.incAndGet();
         }
 
         if (logger.isInfoEnabled())

diff --git a/jvb/src/main/java/org/jitsi/videobridge/Videobridge.java b/jvb/src/main/java/org/jitsi/videobridge/Videobridge.java
@@ -945,18 +945,18 @@ public static class Statistics
         public AtomicInteger incomingBitrateExpirations = new AtomicInteger(0);
 
         /**
-         * The cumulative/total number of conferences that had all of their
-         * channels failed because there was no transport activity (which
-         * includes those that failed because there was no payload activity).
+         * The cumulative/total number of conferences in which ALL of the endpoints failed ICE.
          */
-        public AtomicInteger totalFailedConferences = new AtomicInteger(0);
+        public CounterMetric failedConferences = VideobridgeMetricsContainer.getInstance().registerCounter(
+                "failed_conferences",
+                "Number of conferences in which ALL of the endpoints failed ICE.");
 
         /**
-         * The cumulative/total number of conferences that had some of their
-         * channels failed because there was no transport activity (which
-         * includes those that failed because there was no payload activity).
+         * The cumulative/total number of conferences in which SOME of the endpoints failed ICE.
          */
-        public AtomicInteger totalPartiallyFailedConferences = new AtomicInteger(0);
+        public CounterMetric partiallyFailedConferences = VideobridgeMetricsContainer.getInstance().registerCounter(
+                "partially_failed_conferences",
+                "Number of conferences in which SOME of the endpoints failed ICE.");
 
         /**
          * The cumulative/total number of conferences completed/expired on this

diff --git a/jvb/src/main/java/org/jitsi/videobridge/stats/VideobridgeStatistics.java b/jvb/src/main/java/org/jitsi/videobridge/stats/VideobridgeStatistics.java
@@ -489,10 +489,10 @@ private void generate0()
             unlockedSetStat(RTT_AGGREGATE, rttAggregate);
             unlockedSetStat(
                     TOTAL_FAILED_CONFERENCES,
-                    jvbStats.totalFailedConferences.get());
+                    jvbStats.failedConferences.get());
             unlockedSetStat(
                     TOTAL_PARTIALLY_FAILED_CONFERENCES,
-                    jvbStats.totalPartiallyFailedConferences.get());
+                    jvbStats.partiallyFailedConferences.get());
             unlockedSetStat(
                     TOTAL_CONFERENCES_CREATED,
                     jvbStats.totalConferencesCreated.get());