aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorHarald Musum <musum@verizonmedia.com>2023-02-14 18:50:41 +0100
committerGitHub <noreply@github.com>2023-02-14 18:50:41 +0100
commitbdc3bb0498c334aa70976c876c3451de3ce01251 (patch)
tree51e8ff931089782da9443f5acb36b4a552fc860f
parent707663ec7c53198955868561c512cd805a1cc5ec (diff)
parent48838e42276a7907bba6fa5af40a2d93d420f872 (diff)
Merge pull request #26039 from vespa-engine/hmusum/more-memory-for-arm64-cluster-controllers-in-cdv8.125.29
Give cluster controllers on arm64 in cd more memory
-rw-r--r--node-repository/src/main/java/com/yahoo/vespa/hosted/provision/provisioning/CapacityPolicies.java10
1 files changed, 8 insertions, 2 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 fa265918c33..2a9501e1a8c 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
@@ -9,6 +9,7 @@ import com.yahoo.config.provision.ClusterResources;
import com.yahoo.config.provision.ClusterSpec;
import com.yahoo.config.provision.Environment;
import com.yahoo.config.provision.NodeResources;
+import com.yahoo.config.provision.SystemName;
import com.yahoo.config.provision.Zone;
import com.yahoo.vespa.flags.PermanentFlags;
import com.yahoo.vespa.flags.StringFlag;
@@ -89,7 +90,7 @@ public class CapacityPolicies {
Architecture architecture = adminClusterArchitecture(applicationId);
if (clusterSpec.id().value().equals("cluster-controllers")) {
- return clusterControllerResources(clusterSpec).with(architecture);
+ return clusterControllerResources(clusterSpec, architecture).with(architecture);
}
return (nodeRepository.exclusiveAllocation(clusterSpec)
@@ -111,10 +112,15 @@ public class CapacityPolicies {
}
}
- private NodeResources clusterControllerResources(ClusterSpec clusterSpec) {
+ private NodeResources clusterControllerResources(ClusterSpec clusterSpec, Architecture architecture) {
if (nodeRepository.exclusiveAllocation(clusterSpec)) {
return versioned(clusterSpec, Map.of(new Version(0), smallestExclusiveResources()));
}
+ // Note: Seems like we need more memory for cluster controllers on arm64, trying out in cd
+ if (architecture == Architecture.arm64 && zone.system() == SystemName.cd) {
+ return versioned(clusterSpec, Map.of(new Version(0), new NodeResources(0.25, 1.14, 10, 0.3),
+ new Version(8, 125, 23), new NodeResources(0.25, 1.5, 10, 0.3)));
+ }
return versioned(clusterSpec, Map.of(new Version(0), new NodeResources(0.25, 1.14, 10, 0.3)));
}