summaryrefslogtreecommitdiffstats
path: root/node-admin
diff options
context:
space:
mode:
authorHarald Musum <musum@verizonmedia.com>2019-05-28 10:27:09 +0200
committerHarald Musum <musum@verizonmedia.com>2019-05-28 10:27:09 +0200
commita15dedfcb24b3825710d93e18b7da7001500b4f6 (patch)
treed91c86140bd2244341bef55776ad18c8b175818e /node-admin
parentda58e8e064f7fcbac13836a617100dac2504d607 (diff)
No cpu capping in cd
Diffstat (limited to 'node-admin')
-rw-r--r--node-admin/src/main/java/com/yahoo/vespa/hosted/node/admin/nodeagent/NodeAgentImpl.java9
1 files changed, 8 insertions, 1 deletions
diff --git a/node-admin/src/main/java/com/yahoo/vespa/hosted/node/admin/nodeagent/NodeAgentImpl.java b/node-admin/src/main/java/com/yahoo/vespa/hosted/node/admin/nodeagent/NodeAgentImpl.java
index 8c762736b8a..71710c93742 100644
--- a/node-admin/src/main/java/com/yahoo/vespa/hosted/node/admin/nodeagent/NodeAgentImpl.java
+++ b/node-admin/src/main/java/com/yahoo/vespa/hosted/node/admin/nodeagent/NodeAgentImpl.java
@@ -4,6 +4,8 @@ package com.yahoo.vespa.hosted.node.admin.nodeagent;
import com.fasterxml.jackson.core.JsonProcessingException;
import com.yahoo.config.provision.DockerImage;
import com.yahoo.config.provision.Environment;
+import com.yahoo.config.provision.SystemName;
+import com.yahoo.config.provision.zone.ZoneId;
import com.yahoo.log.LogLevel;
import com.yahoo.vespa.flags.DoubleFlag;
import com.yahoo.vespa.flags.FetchVector;
@@ -361,7 +363,7 @@ public class NodeAgentImpl implements NodeAgent {
}
private ContainerResources getContainerResources(NodeAgentContext context) {
- double cpuCap = context.zoneId().environment() == Environment.dev ?
+ double cpuCap = noCpuCap(context.zoneId()) ?
0 :
context.node().getOwner()
.map(NodeOwner::asApplicationId)
@@ -372,6 +374,11 @@ public class NodeAgentImpl implements NodeAgent {
return ContainerResources.from(cpuCap, context.node().getMinCpuCores(), context.node().getMinMainMemoryAvailableGb());
}
+ private boolean noCpuCap(ZoneId zoneId) {
+ return zoneId.environment() == Environment.dev
+ // TODO: Add other cd systems, not ideal when having just SystemName here
+ || zoneId.system() == SystemName.cd;
+ }
private void scheduleDownLoadIfNeeded(NodeSpec node, Optional<Container> container) {
if (node.getWantedDockerImage().equals(container.map(c -> c.image))) return;