diff options
author | Harald Musum <musum@verizonmedia.com> | 2023-02-14 18:50:41 +0100 |
---|---|---|
committer | GitHub <noreply@github.com> | 2023-02-14 18:50:41 +0100 |
commit | bdc3bb0498c334aa70976c876c3451de3ce01251 (patch) | |
tree | 51e8ff931089782da9443f5acb36b4a552fc860f /node-repository | |
parent | 707663ec7c53198955868561c512cd805a1cc5ec (diff) | |
parent | 48838e42276a7907bba6fa5af40a2d93d420f872 (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
Diffstat (limited to 'node-repository')
-rw-r--r-- | node-repository/src/main/java/com/yahoo/vespa/hosted/provision/provisioning/CapacityPolicies.java | 10 |
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))); } |