Use sentinel value for missing buckets rather than throwing

Bucket count should have been pre-verified as present by the caller.

1 files changed, 1 insertions, 1 deletions

diff --git a/clustercontroller-core/src/main/java/com/yahoo/vespa/clustercontroller/core/NodeStateChangeChecker.java b/clustercontroller-core/src/main/java/com/yahoo/vespa/clustercontroller/core/NodeStateChangeChecker.java
index eed8bc7ce92..814cb48c49f 100644
--- a/clustercontroller-core/src/main/java/com/yahoo/vespa/clustercontroller/core/NodeStateChangeChecker.java
+++ b/clustercontroller-core/src/main/java/com/yahoo/vespa/clustercontroller/core/NodeStateChangeChecker.java
@@ -135,7 +135,7 @@ public class NodeStateChangeChecker {
         long lastEntries = metrics.entries.get().getLast();
         long lastDocs    = metrics.docs.get().getLast();
         if (lastEntries != 0) {
-            long buckets    = metrics.buckets.orElseThrow().getLast();
+            long buckets    = metrics.buckets.map(Metrics.Value::getLast).orElse(-1L);
             long tombstones = lastEntries - lastDocs; // docs are a subset of entries, so |docs| <= |entries|
             return Optional.of(disallow("The storage node stores %d documents and %d tombstones across %d buckets".formatted(lastDocs, tombstones, buckets)));
         }