Remove state tag. Only report metrics for active nodes

1 files changed, 3 insertions, 4 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 a7e82250275..a067fd25753 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,13 +169,13 @@ 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()) {
             ApplicationId applicationId = allocation.get().owner();
             Map<String, String> dimensions = new HashMap<>(dimensions(applicationId));
-            dimensions.put("state", node.state().name());
             dimensions.put("host", node.hostname());
             dimensions.put("clustertype", allocation.get().membership().cluster().type().name());
             dimensions.put("clusterid", allocation.get().membership().cluster().id().value());
@@ -202,8 +202,7 @@ public class MetricsReporter extends NodeRepositoryMaintainer {
                 metric.set(ConfigServerMetrics.HAS_WIRE_GUARD_KEY.baseName(), node.wireguardPubKey().isPresent() ? 1 : 0, context);
             }
         } else {
-            context = getContext(Map.of("state", node.state().name(),
-                                        "host", node.hostname()));
+            context = getContext(Map.of("host", node.hostname()));
         }
 
         Optional<Version> currentVersion = node.status().vespaVersion();