session max duration

Author: SylvainJuge

jmx-scraper/src/integrationTest/java/io/opentelemetry/contrib/jmxscraper/target_systems/JettyIntegrationTest.java

@@ -5,6 +5,7 @@
 
 package io.opentelemetry.contrib.jmxscraper.target_systems;
 
+import static io.opentelemetry.contrib.jmxscraper.target_systems.MetricAssertions.assertGaugeWithAttributes;
 import static io.opentelemetry.contrib.jmxscraper.target_systems.MetricAssertions.assertSumWithAttributes;
 
 import io.opentelemetry.contrib.jmxscraper.JmxScraperContainer;
@@ -63,6 +64,13 @@ protected void verifyMetrics() {
                 "jetty.session.time.total",
                 "The total time sessions have been active.",
                 "s",
+                attrs -> attrs.containsKey("resource")),
+        metric ->
+            assertGaugeWithAttributes(
+                metric,
+                "jetty.session.time.max",
+                "The maximum amount of time a session has been active.",
+                "s",
                 attrs -> attrs.containsKey("resource")));
   }
 }

jmx-scraper/src/main/resources/jetty.yaml

@@ -17,3 +17,10 @@ rules:
         type: counter
         unit: s
         desc: The total time sessions have been active.
+      sessionTimeMax:
+        metric: time.max
+        # here a 'counter' metric type would suit better, but gauge reflects
+        # how it's currently implemented in jetty.groovy
+        type: gauge
+        unit: s
+        desc: The maximum amount of time a session has been active.