diff options
author | gjoranv <gv@yahooinc.com> | 2023-05-05 11:44:15 +0200 |
---|---|---|
committer | gjoranv <gv@yahooinc.com> | 2023-05-05 11:44:15 +0200 |
commit | 7af81bddb9be75b409d108e50c3f79df04bd05e7 (patch) | |
tree | 1aa77e39b6ea93954ffaab49e3ed2ddcee45f3bc | |
parent | f7fd1f95d2de8880dd122efa3f88c2ab1bfd3329 (diff) |
Use constants, and add new metrics to VespaMetricSet.
3 files changed, 11 insertions, 4 deletions
diff --git a/config-model/src/main/java/com/yahoo/vespa/model/admin/monitoring/VespaMetricSet.java b/config-model/src/main/java/com/yahoo/vespa/model/admin/monitoring/VespaMetricSet.java index ac354d66206..f933325b8ae 100644 --- a/config-model/src/main/java/com/yahoo/vespa/model/admin/monitoring/VespaMetricSet.java +++ b/config-model/src/main/java/com/yahoo/vespa/model/admin/monitoring/VespaMetricSet.java @@ -6,12 +6,12 @@ import com.yahoo.metrics.ConfigServerMetrics; import com.yahoo.metrics.ContainerMetrics; import com.yahoo.metrics.DistributorMetrics; import com.yahoo.metrics.LogdMetrics; +import com.yahoo.metrics.NodeAdminMetrics; import com.yahoo.metrics.RoutingLayerMetrics; import com.yahoo.metrics.SearchNodeMetrics; import com.yahoo.metrics.SentinelMetrics; import com.yahoo.metrics.SlobrokMetrics; import com.yahoo.metrics.StorageMetrics; -import com.yahoo.metrics.NodeAdminMetrics; import com.yahoo.metrics.Suffix; import com.yahoo.metrics.VespaMetrics; @@ -239,6 +239,8 @@ public class VespaMetricSet { addMetric(metrics, ContainerMetrics.JDISC_HTTP_HANDLER_UNHANDLED_EXCEPTIONS.rate()); addMetric(metrics, ContainerMetrics.JDISC_APPLICATION_FAILED_COMPONENT_GRAPHS.rate()); + addMetric(metrics, ContainerMetrics.JDISC_APPLICATION_COMPONENT_GRAPH_CREATION_TIME_MILLIS.last()); + addMetric(metrics, ContainerMetrics.JDISC_APPLICATION_COMPONENT_GRAPH_RECONFIGURATIONS.rate()); addMetric(metrics, ContainerMetrics.JDISC_JVM.last()); diff --git a/container-core/src/main/java/com/yahoo/metrics/ContainerMetrics.java b/container-core/src/main/java/com/yahoo/metrics/ContainerMetrics.java index c443c387381..11241e1558d 100644 --- a/container-core/src/main/java/com/yahoo/metrics/ContainerMetrics.java +++ b/container-core/src/main/java/com/yahoo/metrics/ContainerMetrics.java @@ -31,6 +31,8 @@ public enum ContainerMetrics implements VespaMetrics { JDISC_DEACTIVATED_CONTAINERS("jdisc.deactivated_containers.total", Unit.ITEM, "JDISC Deactivated container instances"), JDISC_DEACTIVATED_CONTAINERS_WITH_RETAINED_REFS("jdisc.deactivated_containers.with_retained_refs.last", Unit.ITEM, "JDISC Deactivated container nodes with retained refs"), JDISC_APPLICATION_FAILED_COMPONENT_GRAPHS("jdisc.application.failed_component_graphs", Unit.ITEM, "JDISC Application failed component graphs"), + JDISC_APPLICATION_COMPONENT_GRAPH_CREATION_TIME_MILLIS("jdisc.application.component_graph.creation_time_millis", Unit.MILLISECOND, "JDISC Application component graph creation time"), + JDISC_APPLICATION_COMPONENT_GRAPH_RECONFIGURATIONS("jdisc.application.component_graph.reconfigurations", Unit.ITEM, "JDISC Application component graph reconfigurations"), JDISC_SINGLETON_IS_ACTIVE("jdisc.singleton.is_active", Unit.ITEM, "JDISC Singleton is active"), JDISC_SINGLETON_ACTIVATION_COUNT("jdisc.singleton.activation.count", Unit.OPERATION, "JDISC Singleton activations"), diff --git a/container-disc/src/main/java/com/yahoo/container/jdisc/ConfiguredApplication.java b/container-disc/src/main/java/com/yahoo/container/jdisc/ConfiguredApplication.java index 06aa102b34c..f3f1374762d 100644 --- a/container-disc/src/main/java/com/yahoo/container/jdisc/ConfiguredApplication.java +++ b/container-disc/src/main/java/com/yahoo/container/jdisc/ConfiguredApplication.java @@ -65,6 +65,9 @@ import java.util.logging.Logger; import static com.yahoo.collections.CollectionUtil.first; import static com.yahoo.metrics.ContainerMetrics.APPLICATION_GENERATION; +import static com.yahoo.metrics.ContainerMetrics.JDISC_APPLICATION_COMPONENT_GRAPH_CREATION_TIME_MILLIS; +import static com.yahoo.metrics.ContainerMetrics.JDISC_APPLICATION_COMPONENT_GRAPH_RECONFIGURATIONS; +import static com.yahoo.metrics.ContainerMetrics.JDISC_APPLICATION_FAILED_COMPONENT_GRAPHS; /** * @author Tony Vaagenes @@ -341,8 +344,8 @@ public final class ConfiguredApplication implements Application { Runnable cleanupTask = configurer.waitForNextGraphGeneration(builder.guiceModules().activate(), false); initializeAndActivateContainer(builder, cleanupTask); var metric = configurer.getComponent(Metric.class); - metric.set("jdisc.application.component_graph.creation_time_millis", Duration.between(start, Instant.now()).toMillis(), null); - metric.add("jdisc.application.component_graph.reconfigurations", 1L, null); + metric.set(JDISC_APPLICATION_COMPONENT_GRAPH_CREATION_TIME_MILLIS.baseName(), Duration.between(start, Instant.now()).toMillis(), null); + metric.add(JDISC_APPLICATION_COMPONENT_GRAPH_RECONFIGURATIONS.baseName(), 1L, null); } catch (UncheckedInterruptedException | SubscriberClosedException | ConfigInterruptedException e) { break; } catch (Exception | LinkageError e) { // LinkageError: OSGi problems @@ -364,7 +367,7 @@ public final class ConfiguredApplication implements Application { // Metric may not be available if this is the initial component graph (since metric wiring is done through the config model) Metric metric = configurer.getComponent(Metric.class); Metric.Context metricContext = metric.createContext(Map.of("exception", error.getClass().getSimpleName())); - metric.add("jdisc.application.failed_component_graphs", 1L, metricContext); + metric.add(JDISC_APPLICATION_FAILED_COMPONENT_GRAPHS.baseName(), 1L, metricContext); } catch (Exception e) { log.log(Level.WARNING, "Failed to report metric for failed component graph: " + e.getMessage(), e); } |