diff options
author | Ola Aunrønning <olaa@vespa.ai> | 2024-03-27 14:32:33 +0100 |
---|---|---|
committer | GitHub <noreply@github.com> | 2024-03-27 14:32:33 +0100 |
commit | 4763e1a2e98ca25c9e08d1d3516a721e9b1dbcc6 (patch) | |
tree | 146d5363d0ca7df32a0d232e13a876513cd143fc /node-repository | |
parent | 680f29db27d18fa1c2c957b92fb8bb2c67509dc6 (diff) | |
parent | 4a022158a8d2d500f43b8bb271ad339fd75895b3 (diff) |
Merge pull request #30744 from vespa-engine/olaa/nonactive-node-metrics
Emit suspended seconds. Update metrics for non-active nodes
Diffstat (limited to 'node-repository')
-rw-r--r-- | node-repository/src/main/java/com/yahoo/vespa/hosted/provision/maintenance/MetricsReporter.java | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/maintenance/MetricsReporter.java b/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/maintenance/MetricsReporter.java index 69ae4fddb63..e3d72d1189e 100644 --- a/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/maintenance/MetricsReporter.java +++ b/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/maintenance/MetricsReporter.java @@ -169,8 +169,6 @@ public class MetricsReporter extends NodeRepositoryMaintainer { * NB: Keep this metric set in sync with internal configserver metric pre-aggregation */ private void updateNodeMetrics(Node node, ServiceModel serviceModel) { - if (node.state() != State.active) - return; Metric.Context context; Optional<Allocation> allocation = node.allocation(); if (allocation.isPresent()) { @@ -235,7 +233,7 @@ public class MetricsReporter extends NodeRepositoryMaintainer { long suspendedSeconds = info.suspendedSince() .map(suspendedSince -> Duration.between(suspendedSince, clock().instant()).getSeconds()) .orElse(0L); - metric.set(ConfigServerMetrics.SUSPENDED_SECONDS.baseName(), suspendedSeconds, context); + metric.add(ConfigServerMetrics.SUSPENDED_SECONDS.baseName(), suspendedSeconds, context); }); long numberOfServices; |