diff options
author | Jon Bratseth <bratseth@gmail.com> | 2022-04-12 18:25:00 +0200 |
---|---|---|
committer | GitHub <noreply@github.com> | 2022-04-12 18:25:00 +0200 |
commit | 6e15dfa09d2fcdf9470fa873ec6f256ff7c42bb3 (patch) | |
tree | 2fdd489d7ea366bcaac79c994b392b2f4fe87acc /node-repository/src/main/java/com/yahoo/vespa/hosted/provision/maintenance/MetricsReporter.java | |
parent | 4f22734462a0fac9dcc4fcdc8542218bb849c842 (diff) |
Revert "Preserve all node events"
Diffstat (limited to 'node-repository/src/main/java/com/yahoo/vespa/hosted/provision/maintenance/MetricsReporter.java')
-rw-r--r-- | node-repository/src/main/java/com/yahoo/vespa/hosted/provision/maintenance/MetricsReporter.java | 5 |
1 files changed, 3 insertions, 2 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 b117523dce0..1fe29c8b162 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 @@ -20,6 +20,7 @@ import com.yahoo.vespa.hosted.provision.NodeList; import com.yahoo.vespa.hosted.provision.NodeRepository; import com.yahoo.vespa.hosted.provision.node.Allocation; import com.yahoo.vespa.hosted.provision.node.ClusterId; +import com.yahoo.vespa.hosted.provision.node.History; import com.yahoo.vespa.hosted.provision.persistence.CacheStats; import com.yahoo.vespa.service.monitor.ServiceModel; import com.yahoo.vespa.service.monitor.ServiceMonitor; @@ -247,8 +248,8 @@ public class MetricsReporter extends NodeRepositoryMaintainer { boolean down = NodeHealthTracker.allDown(services); metric.set("nodeFailerBadNode", (down ? 1 : 0), context); - boolean recordedDown = node.isDown(); - metric.set("downInNodeRepo", (recordedDown ? 1 : 0), context); + boolean nodeDownInNodeRepo = node.history().event(History.Event.Type.down).isPresent(); + metric.set("downInNodeRepo", (nodeDownInNodeRepo ? 1 : 0), context); } metric.set("numberOfServices", numberOfServices, context); |