Filter out NaN values

1 files changed, 8 insertions, 1 deletions

diff --git a/docker-api/src/main/java/com/yahoo/vespa/hosted/dockerapi/metrics/DimensionMetrics.java b/docker-api/src/main/java/com/yahoo/vespa/hosted/dockerapi/metrics/DimensionMetrics.java
index ef59c4b17d6..f13ef896b7f 100644
--- a/docker-api/src/main/java/com/yahoo/vespa/hosted/dockerapi/metrics/DimensionMetrics.java
+++ b/docker-api/src/main/java/com/yahoo/vespa/hosted/dockerapi/metrics/DimensionMetrics.java
@@ -9,6 +9,7 @@ import java.util.List;
 import java.util.Map;
 import java.util.Objects;
 import java.util.TreeMap;
+import java.util.stream.Collectors;
 
 /**
  * @author freva
@@ -24,7 +25,9 @@ public class DimensionMetrics {
     DimensionMetrics(String application, Dimensions dimensions, Map<String, Number> metrics) {
         this.application = Objects.requireNonNull(application);
         this.dimensions = Objects.requireNonNull(dimensions);
-        this.metrics = Objects.requireNonNull(metrics);
+        this.metrics = metrics.entrySet().stream()
+                .filter(DimensionMetrics::metricIsNotNaN)
+                .collect(Collectors.toMap(Map.Entry::getKey, Map.Entry::getValue));
     }
 
     public String toSecretAgentReport() throws JsonProcessingException {
@@ -65,6 +68,10 @@ public class DimensionMetrics {
         return Objects.hash(application, dimensions, metrics);
     }
 
+    private static boolean metricIsNotNaN(Map.Entry<String, Number> metric) {
+        return ! (metric.getValue() instanceof Double && Double.isNaN((double) metric.getValue()));
+    }
+
     public static class Builder {
         private final String application;
         private final Dimensions dimensions;