summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorJon Bratseth <bratseth@verizonmedia.com>2019-05-21 11:54:32 +0200
committerJon Bratseth <bratseth@verizonmedia.com>2019-05-21 11:54:32 +0200
commite5aaec53b70207064bb78c8136656a580b135166 (patch)
treedb7825dda7fcae4cb8c72ee7268ee9c04d336863
parente3d08bf6f8ee38c1242d87f09d97323c7e07dfdd (diff)
Correct to any disk in CD for default flavors
CD specifies docker default flavors, but docker flavors implies fast disk, which is absent in CD.
-rw-r--r--node-repository/src/main/java/com/yahoo/vespa/hosted/provision/provisioning/CapacityPolicies.java10
1 files changed, 7 insertions, 3 deletions
diff --git a/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/provisioning/CapacityPolicies.java b/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/provisioning/CapacityPolicies.java
index 349be09c4a5..d36638a1a66 100644
--- a/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/provisioning/CapacityPolicies.java
+++ b/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/provisioning/CapacityPolicies.java
@@ -42,10 +42,14 @@ public class CapacityPolicies {
}
public NodeResources decideNodeResources(Capacity requestedCapacity, ClusterSpec cluster) {
- Optional<NodeResources> requestedResources = requestedCapacity.nodeResources();
+ NodeResources resources = decideNodeResources(requestedCapacity.nodeResources(), cluster);
+ if (zone.system() == SystemName.cd)
+ return resources.withDiskSpeed(NodeResources.DiskSpeed.any);
+ else
+ return resources;
+ }
- if (zone.system() == SystemName.cd && requestedResources.isPresent())
- requestedResources = Optional.of(requestedResources.get().withDiskSpeed(NodeResources.DiskSpeed.any));
+ private NodeResources decideNodeResources(Optional<NodeResources> requestedResources, ClusterSpec cluster) {
if (requestedResources.isPresent() && ! requestedResources.get().allocateByLegacyName())
return requestedResources.get();