diff options
Diffstat (limited to 'controller-server/src/main/java/com/yahoo/vespa/hosted/controller/versions/OsVersionStatus.java')
-rw-r--r-- | controller-server/src/main/java/com/yahoo/vespa/hosted/controller/versions/OsVersionStatus.java | 15 |
1 files changed, 9 insertions, 6 deletions
diff --git a/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/versions/OsVersionStatus.java b/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/versions/OsVersionStatus.java index 6f9888b79e0..a06f26b1fdf 100644 --- a/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/versions/OsVersionStatus.java +++ b/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/versions/OsVersionStatus.java @@ -14,6 +14,7 @@ import com.yahoo.vespa.hosted.controller.maintenance.OsUpgrader; import java.time.Instant; import java.util.ArrayList; import java.util.Collection; +import java.util.Collections; import java.util.HashMap; import java.util.List; import java.util.Map; @@ -36,13 +37,15 @@ public record OsVersionStatus(Map<OsVersion, List<NodeVersion>> versions) { this.versions = ImmutableMap.copyOf(Objects.requireNonNull(versions, "versions must be non-null")); } - /** Returns nodes eligible for OS upgrades that exist in given cloud */ + /** Returns all node versions that exist in given cloud */ public List<NodeVersion> nodesIn(CloudName cloud) { - return versions.entrySet().stream() - .filter(entry -> entry.getKey().cloud().equals(cloud)) - .map(Map.Entry::getValue) - .findFirst() - .orElseGet(List::of); + List<NodeVersion> nodeVersions = new ArrayList<>(); + versions.forEach((osVersion, nodesOnVersion) -> { + if (osVersion.cloud().equals(cloud)) { + nodeVersions.addAll(nodesOnVersion); + } + }); + return Collections.unmodifiableList(nodeVersions); } /** Returns versions that exist in given cloud */ |